JerryYue-M opened a new pull request #5057:
URL: https://github.com/apache/hudi/pull/5057


   currently. HoodieHiveClient both have DDLExecutor and IMetaStoreClient 
instance, some ddl executor implemented such as HMSDDLExecutor and 
HiveQueryDDLExecutor will create IMetaStoreClient instance again. some method 
in HoodieHiveClient operate HMS both use ddl executor and IMetaStoreClient 
instance. it look not a clear style. we do some change here.
   
   1.HoodieHiveClient operate HMS only via DDLExecutor instance.
   
   2.Only create IMetaStoreClient instance one time, and pass to ddl executor
   
   3.no need create partitionValueExtractor in HoodieHiveClient, use 
DDLExecutor's partitionValueExtractor
   
   4.no need public declaration of interface(DDLExecutor) method ,remove it
   
   5.remove some unused import


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