[GitHub] [arrow] pitrou commented on a change in pull request #10997: ARROW-13218: [Format] Clarify interpretation of timestamp values

[GitBox <gi...@apache.org>]

pitrou commented on a change in pull request #10997:
URL: https://github.com/apache/arrow/pull/10997#discussion_r695869008



##########
File path: format/Schema.fbs
##########
@@ -214,58 +214,106 @@ table Time {
   bitWidth: int = 32;
 }
 
-/// Time elapsed from the Unix epoch, 00:00:00.000 on 1 January 1970, excluding
-/// leap seconds, as a 64-bit integer. Note that UNIX time does not include
-/// leap seconds.
+/// Timestamp is a 64-bit signed integer representing an elapsed time since a
+/// fixed epoch, stored in either of four units: seconds, milliseconds,
+/// microseconds or nanoseconds, and interpretable in a particular timezone.

Review comment:
       I will commit an alternative wording:
   > Timestamp is a 64-bit signed integer representing an elapsed time since a fixed epoch, stored in either of four units: seconds, milliseconds, microseconds or nanoseconds, and is optionally annotated with a timezone.




-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org