peterxcli opened a new pull request, #5133: URL: https://github.com/apache/datafusion-comet/pull/5133
## Which issue does this PR close? Closes #5058. ## Rationale for this change `CalendarIntervalType` is supported as Arrow `Interval(MonthDayNano)`, but the native planner did not construct a null scalar for that type. As a result, null calendar interval literals passed the JVM support check and then failed during native planning. ## What changes are included in this PR? Map null `Interval(MonthDayNano)` literals to `ScalarValue::IntervalMonthDayNano(None)` in the native planner. ## How are these changes tested? The existing `calendar_interval.sql` fixture covers `CAST(NULL AS INTERVAL)` and requires native execution. - `cargo fmt --manifest-path native/Cargo.toml --all --check` - `make core` - `./mvnw test -Dtest=none -Dsuites="org.apache.comet.CometSqlFileTestSuite calendar_interval" -Dscalastyle.skip=true` with JDK 17 (1 test passed) -- 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]
