[GitHub] [iceberg] pvary commented on pull request #4734: ORC: Upgrade orc version to use the built-in estimate memory method

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

pvary commented on PR #4734:
URL: https://github.com/apache/iceberg/pull/4734#issuecomment-1142261908

   > Do we shade in the ORC Version? Just checking because this could potentially break folks who are using the older versions of ORC at runtime right?
   
   I think we do not shade things. We had to do that in Hive code to avoid clashing versions. We also struggle with Parquet and Avro. It might worth considering shading our own versions of file formats. But it is for another PR.


-- 
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@iceberg.apache.org

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


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