ganeshashree commented on PR #4444: URL: https://github.com/apache/hive/pull/4444#issuecomment-1612522889
@okumin Thank you for addressing this! I vote for users to specify an implementation of `IMetaStoreClient` via HiveConf and use that in `SessionHiveMetastoreClient` instead of specifying a factory class. This way, it will be easy for users to just implement their version of `IMetaStoreClient` and also get all the features implemented in `SessionHiveMetastoreClient` along with `RetryingMetaStoreClient`. I am also thinking of using the specified implementation of `IMetaStoreClient` for delegating the calls in `SessionHiveMetastoreClient` (same as you mentioned). I would let senior community members to review this approach and give their feedback. -- 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]
