Re: [PR] GH-3083: Make DELTA_LENGTH_BYTE_ARRAY default encoding for binary [parquet-java]

["jhorstmann (via GitHub)" <gi...@apache.org>]

jhorstmann commented on PR #3085:
URL: https://github.com/apache/parquet-java/pull/3085#issuecomment-2598424757

   In my opinion the default should not be changed, but it would be really useful to allow users to configure the encoding per column, similar to how `BYTE_STREAM_SPLIT` is now handled in [`ParquetProperties`](https://github.com/apache/parquet-java/blob/master/parquet-column/src/main/java/org/apache/parquet/column/ParquetProperties.java#L500).
   
   There might still be parquet readers with incomplete support for delta encodings, a simple version update could otherwise lead to problems which are only noticed much later.


-- 
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: issues-unsubscribe@parquet.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@parquet.apache.org
For additional commands, e-mail: issues-help@parquet.apache.org