[
https://issues.apache.org/jira/browse/HUDI-3651?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
yuemeng updated HUDI-3651:
--------------------------
Description:
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
was:
currently. HoodieHiveClient both have DDLExecutor and IMetaStoreClient
instance, some ddl executor implemented such as HMSDDLExecutor,
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
> optimize the hoodie hive client and ddl executor code with simpler and
> clearer abstraction
> ------------------------------------------------------------------------------------------
>
> Key: HUDI-3651
> URL: https://issues.apache.org/jira/browse/HUDI-3651
> Project: Apache Hudi
> Issue Type: Improvement
> Reporter: yuemeng
> Priority: Major
> Labels: pull-request-available
>
> 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 message was sent by Atlassian Jira
(v8.20.1#820001)
