fix: attempt to divide by zero error on decimal division

[viirya]

Which issue does this PR close?

Closes #171.

Rationale for this change

What changes are included in this PR?

How are these changes tested?

[viirya]

cc @sunchao

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 33.29%. Comparing base (9a58880) to head (2d7fac1).
Report is 2 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main     #172      +/-   ##
============================================
+ Coverage     33.27%   33.29%   +0.01%     
- Complexity      763      766       +3     
============================================
  Files           105      107       +2     
  Lines         35373    35379       +6     
  Branches       7663     7657       -6     
============================================
+ Hits          11772    11778       +6     
- Misses        21151    21155       +4     
+ Partials       2450     2446       -4     

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

[viirya]

cc @kazuyukitanimura

[kazuyukitanimura]

LGTM
I wonder if we need a similar fix for modulo operation

[viirya]

LGTM I wonder if we need a similar fix for modulo operation

Thanks for review.

I think for modulo operation, we don't implement the operation in Comet but delegate to DataFusion?
I just tested it and looks okay. Let me also add it in the test.

[kazuyukitanimura]

Great, thanks for confirming

[sunchao]

LGTM

[viirya]

Merged. Thanks.