You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by "jnturton (via GitHub)" <gi...@apache.org> on 2023/02/01 14:43:35 UTC

[GitHub] [drill] jnturton commented on issue #2746: [DISCUSSION] Use INT96 as default timestamp format in Parquet tables

jnturton commented on issue #2746:
URL: https://github.com/apache/drill/issues/2746#issuecomment-1412173964

   I'm a -1 at this point. Even though it has historically been a popular thing to do, the use of INT96 for timestamps is nonstandard and deprecated [1]. The Parquet project wants to discourage its ongoing use [2].
   
   1. https://github.com/apache/parquet-format/pull/86/files
   2. https://issues.apache.org/jira/browse/PARQUET-323
   
   So I think we should leave this switched off by default and change the description of our option to say that INT96 timestamps are deprecated and we only recommend enabling them for the sake of reading legacy data.


-- 
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: dev-unsubscribe@drill.apache.org

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