stream2000 commented on code in PR #10048:
URL: https://github.com/apache/hudi/pull/10048#discussion_r1390780089


##########
hudi-utilities/src/main/java/org/apache/hudi/utilities/multitable/MultiTableServiceUtils.java:
##########
@@ -166,6 +168,9 @@ public static TableServicePipeline 
buildTableServicePipeline(JavaSparkContext js
                                                                
HoodieMultiTableServicesMain.Config cfg,
                                                                TypedProperties 
props) {
     TableServicePipeline pipeline = new TableServicePipeline();
+    HoodieTableMetaClient metaClient = UtilHelpers.createMetaClient(jsc, 
basePath, true);
+    // Add the table config to the write config.
+    props.putAll(metaClient.getTableConfig().getProps());

Review Comment:
   Should we use table config to overwrite those configs already set in write 
configs by user?  Not sure which one have a higher priority here. @danny0405 
What do you think? 



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