peterxcli opened a new pull request, #5891:
URL: https://github.com/apache/datafusion-comet/pull/5891
## Which issue does this PR close?
Closes #5517.
## Rationale for this change
Parquet TIMESTAMP_MILLIS overflow currently surfaces as a raw Arrow error.
Spark throws a plain `ArithmeticException("long overflow")`, even with ANSI
disabled.
## What changes are included in this PR?
Emit `SparkError::LongOverflow` from checked millisecond-to-microsecond
conversion and translate it in the shared JVM error converter. The shared error
variant, mapping, and tests match #5457; whichever PR lands second can drop the
overlapping additions on rebase.
## How are these changes tested?
Regression tests check the exact exception class and message for positive
and negative overflow, dictionary pages, nested timestamps, and both ANSI
settings. Native tests cover JSON serialization and null visibility; existing
filtered-scan controls also pass. Focused checks pass on Spark 3.4, 3.5, and
4.1.
--
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]
