Re: Discussion about HIVE-12679 to make IMetaStoreClient pluggable

2023-10-19 Thread Stamatis Zampetakis
Hey Okumin, Thanks for picking up this ticket and driving it forward. I don't have a strong opinion between the two options. On the surface the factory option seems simpler and possibly more efficient but I am not sure if the changes under the PR are sufficient to cover all usages in Hive. On t

Re: Discussion about HIVE-12679 to make IMetaStoreClient pluggable

2023-10-19 Thread Attila Turoczy
Hi All, In my mind, the proxy option is to encapsulate the implementation and more likely force the IMetaStoreClient to have understanding about the RetryingMetaStoreClient or HiveMetaStoreClientWithLocalCache. Which should not be necessary all the time. IMetaStoreClient should be just an abstrac