Skip to content

fix: normalize negative-scale BigDecimal parameters before Arrow encoding - #202

Merged
KaviarasuSakthivadivel merged 4 commits into
mainfrom
fix/negative-scale-decimal-arrow-param
Sep 30, 2026
Merged

KaviarasuSakthivadivel merged 4 commits into
mainfrom
fix/negative-scale-decimal-arrow-param

fix: also widen precision when a BigDecimal parameter's scale exceeds it

4eff1b5
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Sep 29, 2026 in 0s

81.11% (+0.12%) compared to 80ee690

View this Pull Request on Codecov

81.11% (+0.12%) compared to 80ee690

Details

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.11%. Comparing base (80ee690) to head (4eff1b5).

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #202      +/-   ##
============================================
+ Coverage     80.99%   81.11%   +0.12%     
- Complexity     1726     1735       +9     
============================================
  Files           123      123              
  Lines          5051     5063      +12     
  Branches        524      526       +2     
============================================
+ Hits           4091     4107      +16     
+ Misses          728      726       -2     
+ Partials        232      230       -2     
Components Coverage Δ
JDBC Core 81.14% <100.00%> (+0.15%) ⬆️
JDBC Main 57.42% <ø> (ø)
JDBC HTTP 90.71% <ø> (ø)
JDBC Utilities 65.25% <ø> (ø)
Spark Datasource ∅ <ø> (∅)
Files with missing lines Coverage Δ
...force/datacloud/jdbc/protocol/data/ArrowUtils.java 82.97% <100.00%> (+11.55%) ⬆️

... and 1 file with indirect coverage changes

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.