xiangfu0 opened a new pull request, #17677:
URL: https://github.com/apache/pinot/pull/17677
## Summary
This extracts the production truncate(value) behavior fix from #17652 into a
standalone PR, per review feedback.
### Changes
- Replace per-row BigDecimal allocation in the no-scale path with
allocation-free math truncation (floor/ceil)
- Preserve historical behavior for non-finite inputs (NaN, +Infinity,
-Infinity) in the no-scale path
- Canonicalize zero to +0.0 for deterministic output (-0.0 -> +0.0)
- Extend tests with:
- signed-zero bit-level regression coverage
- explicit no-scale non-finite coverage
## Validation
- ./mvnw -pl pinot-core -am -Dtest=TruncateDecimalTransformFunctionTest
-Dsurefire.failIfNoSpecifiedTests=false test
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
