nandorKollar commented on PR #3741:
URL: https://github.com/apache/polaris/pull/3741#issuecomment-3887394009

   > @nandorKollar FYI, Iceberg is still using Hive 2, I'm not sure there's any 
plan to upgrade HiveCatalog in Iceberg. Maybe the Hive community can "host" the 
HiveCatalog and update it.
   
   Actually it seems that Hive already took over the Iceberg related 
integration parts, including the HiveCatalog: 
https://github.com/apache/hive/blob/master/iceberg/iceberg-catalog/src/main/java/org/apache/iceberg/hive/HiveCatalog.java
   
   We should eventually use this one, instead of one in Iceberg code. I tried 
to replace it, but hit an exception, which requires further debugging. I agree 
with you, probably better to find the problematic transitive dependency for 
Hive 3, and try to exclude it if possible, and handle Hive 4 in a separate 
change.


-- 
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]

Reply via email to