fix: Fix overflow handling when casting float to decimal

[leung-ming]

Which issue does this PR close?

Rationale for this change

The continue after append_null was forgotten

What changes are included in this PR?

Add continue

How are these changes tested?

org.apache.comet.CometCastSuite

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.96%. Comparing base (f09f8af) to head (eb3953e).
Report is 282 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #1914      +/-   ##
============================================
+ Coverage     56.12%   58.96%   +2.83%     
- Complexity      976     1141     +165     
============================================
  Files           119      130      +11     
  Lines         11743    12865    +1122     
  Branches       2251     2420     +169     
============================================
+ Hits           6591     7586     +995     
- Misses         4012     4059      +47     
- Partials       1140     1220      +80     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[parthchandra]

Would you like to add a unit test to cover this case?

[leung-ming]

Would you like to add a unit test to cover this case?

Ok, do it later.

[parthchandra]

Thanks @leung-ming, for the test on the native side. I was really thinking of a unit test in org.apache.comet.CometCastSuite

[andygrove]

The CI test failure is unrelated to changes in this PR and is now fixed in main branch

[andygrove]

The CI failure can be fixed by merging latest from main branch.

[andygrove]

Thanks @leung-ming