Skip to content

Decimal arithmetic dtypes and widening rescale cast#8343

Open
gatesn wants to merge 1 commit into
developfrom
ngates/decimal-arithmetic-dtypes
Open

Decimal arithmetic dtypes and widening rescale cast#8343
gatesn wants to merge 1 commit into
developfrom
ngates/decimal-arithmetic-dtypes

Conversation

@gatesn

@gatesn gatesn commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Binary::return_dtype derives Hive-style result types for decimal + - * / (mirroring arrow-arith's execution-time rules, with precision saturation at the physical width), instead of erroring at plan time. The decimal cast kernel gains widening rescale so expression coercion can align decimal operand types.

Binary::return_dtype derives Hive-style result types for decimal
+ - * / (mirroring arrow-arith's execution-time rules, with precision
saturation at the physical width), instead of erroring at plan time.
The decimal cast kernel gains widening rescale so expression coercion
can align decimal operand types.

Needed by vortex-engine for TPC-H Q1/Q6 decimal expressions.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Signed-off-by: Nicholas Gates <nick@nickgates.com>
@gatesn gatesn requested a review from AdamGS June 10, 2026 20:07
@gatesn gatesn added the changelog/fix A bug fix label Jun 10, 2026
@gatesn gatesn marked this pull request as ready for review June 10, 2026 20:07
@gatesn gatesn requested a review from a team June 10, 2026 20:07
@gatesn gatesn enabled auto-merge (squash) June 10, 2026 20:07
@codspeed-hq

codspeed-hq Bot commented Jun 10, 2026

Copy link
Copy Markdown

Merging this PR will degrade performance by 27.11%

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

❌ 3 regressed benchmarks
✅ 1529 untouched benchmarks

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation chunked_bool_canonical_into[(1000, 10)] 20.2 µs 35.5 µs -43.03%
Simulation chunked_varbinview_canonical_into[(1000, 10)] 162 µs 198.1 µs -18.25%
Simulation chunked_varbinview_into_canonical[(1000, 10)] 177.2 µs 213.1 µs -16.86%

Tip

Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.


Comparing ngates/decimal-arithmetic-dtypes (2c37139) with develop (3d7bbfb)

Open in CodSpeed

@AdamGS AdamGS self-assigned this Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/fix A bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants