gszadovszky commented on pull request #831: URL: https://github.com/apache/parquet-mr/pull/831#issuecomment-737864597
Parquet community was against adding INT96 support to not to encourage our clients to use it. While I understand the requirement of supporting the already written types. (Meanwhile as parquet-avro did not support INT96 ever this change is required for developments of new functionalities depending on the deprecated INT96.) Anyway, I am fine with this change but I do not really like that it works by default. What do you think about keeping the original behavior by default and introduce a configuration flag to switch it on? (See `writeParquetUUID` as an example.) This way we still not encourage the clients to use INT96 but have the option to do so if it is necessary. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
