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

   > @pvary multiple version HMS support is only applicable for Spark's session 
catalog HMS client, Iceberg Hive Catalog manages its own HMS client and always 
uses Spark's built-in Hive classes.
   
   I don't think `always uses Spark's built-in Hive classes` is true. Maybe, if 
Spark does some magic with the classloader and prevents using the classes 
coming from Iceberg. The Iceberg Hive Catalog uses whatever HMS client is on 
the classpath and finds first. In this case I'm not sure why do we embed any 
Hive jars to the Spark runtime jar as it would be ignored and the built in 
classes would be used.
   
   Anyways, this is deeper than my Spark classpath handling reaches, so I would 
wait for others with more Spark knowledge to chime in. 
   


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

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


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

Reply via email to