MaxGekk commented on code in PR #56320: URL: https://github.com/apache/spark/pull/56320#discussion_r3363098039
########## sql/core/src/test/resources/sql-tests/inputs/timestamp-ltz-nanos.sql: ########## Review Comment: Added. Note: NULLs for these columns are handled by the generic `(null, _)` branch in `HiveResult.toHiveString`, which runs before the new `TimestampLTZNanosType` / `TimestampNTZNanosType` cases, so the path is type-agnostic -- but it's worth locking in. Added top-level (`NULL`) and nested array/map/struct NULL cases to both golden files (and the suite). Done in d578fbba68f. -- 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]
