szehon-ho opened a new pull request #3403:
URL: https://github.com/apache/iceberg/pull/3403


   Fixes : https://github.com/apache/iceberg/issues/3363, 
RetryingMetaStoreClient broken for Hive 2.1.
   
   The root cause is this commit: 
https://issues.apache.org/jira/browse/HIVE-12918, fixed in Hive 2.3 (the 
version we build against) by https://issues.apache.org/jira/browse/HIVE-15081
   
   Trying to use this more stable method in RetryingMetaStoreClient: 
https://github.com/apache/hive/blob/rel/release-2.3.8/metastore/src/java/org/apache/hadoop/hive/metastore/RetryingMetaStoreClient.java#L95.
  
   
   I'm usually hesitant to use @VisibleForTesting methods, but as @sunchao 
pointed out offline to me, for Hive it means the method is at least tested :), 
unlike the non marked one.


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



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to