alamb commented on code in PR #26867: URL: https://github.com/apache/datafusion/pull/26867#discussion_r4415097004
########## datafusion/sqllogictest/test_files/datetime/timestamps_timezone.slt: ########## @@ -1377,33 +1377,27 @@ Timestamp(ns, "+00:00") 2024-07-01T06:00:00Z ########## ## SECTION 16: DST boundaries ## -## KNOWN WRONG, and the single largest divergence in this file: DataFusion -## cannot represent an ambiguous or non-existent local time in a named zone at -## all. Every route into a named zone -- a column cast, `AT TIME ZONE`, a -## string literal, or `::timestamptz` under a session zone -- fails with an -## error, where PostgreSQL resolves the value. -## -## PostgreSQL's answers, for reference: +## Casting a naive timestamp into a named zone (a column cast or +## `AT TIME ZONE`) resolves an ambiguous or non-existent local time the way +## PostgreSQL does, since https://github.com/apache/arrow-rs/pull/11038: Review Comment: Due to - https://github.com/apache/arrow-rs/pull/11038 ########## datafusion/sqllogictest/test_files/datetime/timestamps_timezone.slt: ########## @@ -1377,33 +1377,24 @@ Timestamp(ns, "+00:00") 2024-07-01T06:00:00Z ########## ## SECTION 16: DST boundaries ## -## KNOWN WRONG, and the single largest divergence in this file: DataFusion -## cannot represent an ambiguous or non-existent local time in a named zone at -## all. Every route into a named zone -- a column cast, `AT TIME ZONE`, a -## string literal, or `::timestamptz` under a session zone -- fails with an -## error, where PostgreSQL resolves the value. +## Casting a naive timestamp into a named zone resolves an ambiguous or Review Comment: Due to - https://github.com/apache/arrow-rs/pull/11038 -- 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]
