YutaLin opened a new issue, #4396: URL: https://github.com/apache/datafusion-comet/issues/4396
### What is the problem the feature request solves? This is https://github.com/apache/datafusion-comet/issues/4288 sub-issue These are the core infrastructure pieces that make TimeType columns usable in real-world queries. Without them, any query that shuffles, sorts, or aggregates a time column falls back to Spark. Since TimeType is physically a fixed-width i64 (same as LongType/TimestampType), all of these should be straightforward additions to existing type lists. ### Describe the potential solution **1a. Shuffle (native)** File: spark/.../CometShuffleExchangeExec.scala:367-380 supportedSerializableDataType does not include TimeType. Add it alongside the existing TimestampType | TimestampNTZType case. Also need to check the Rust shuffle path: native/shuffle/src/spark_unsafe/row.rs - needs Time64 handling **1b. Shuffle (JVM columnar)** File: spark/.../CometShuffleExchangeExec.scala:490-506 Same supportedSerializableDataType function (different scope). Add TimeType. ### Additional context _No response_ -- 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]
