RyanSkraba commented on PR #2554: URL: https://github.com/apache/avro/pull/2554#issuecomment-1783271756
Hello! I just wanted to add some reference materials that might be useful to understanding where we are today! 1. The AVRO `local-timestamp-x` design documents are at https://issues.apache.org/jira/browse/AVRO-2328 2. [This document](https://docs.google.com/document/d/1E-7miCh4qK6Mg54b-Dh5VOyhGX8V4xdMXKIHJL36a9U/edit) (especially Appendix 2) gives some pretty compelling arguments to including the `local-timestamp-x` types. 3. It builds on another, [more detailed document](https://docs.google.com/document/d/1gNRww9mZJcHvUDCXklzjFEQGpefsuR_akCDfWsdE35Q/edit) "Consistent timestamp types in Hadoop SQL engines" that has a wider scope across Big Data SQL engines. (I refer to these document more than I care to admit...) My subjective opinion is that I'd prefer to resist adding new LogicalTypes, but this is a pretty compelling case. I really don't see any disadvantages to extending the existing timestamp types with more precision. Any SDK that can't or won't implement them will just have the nanos `int64` to fall back on without any loss of precision. -- 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]
