jorgecarleitao commented on a change in pull request #11138:
URL: https://github.com/apache/arrow/pull/11138#discussion_r708018490
##########
File path: format/Schema.fbs
##########
@@ -357,18 +357,23 @@ enum IntervalUnit: short { YEAR_MONTH, DAY_TIME,
MONTH_DAY_NANO}
// days can differ in length during day light savings time transitions).
// All integers in the types below are stored in the endianness indicated
// by the schema.
+//
// YEAR_MONTH - Indicates the number of elapsed whole months, stored as
// 4-byte signed integers.
-// DAY_TIME - Indicates the number of elapsed days and milliseconds,
-// stored as 2 contiguous 32-bit integers (8-bytes in total). Support
+// DAY_TIME - Indicates the number of elapsed days and milliseconds (no leap
seconds),
+// stored as 2 contiguous 32-bit integers (8-bytes in total). Support
// of this IntervalUnit is not required for full arrow compatibility.
+// DAY_TIME interval doesn't have total order due to day light savings.
Review comment:
maybe removing this sentence to not discuss ordering?
--
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]
