leesf commented on a change in pull request #3427:
URL: https://github.com/apache/hudi/pull/3427#discussion_r686948797



##########
File path: 
hudi-common/src/main/java/org/apache/hudi/common/config/HoodieMetadataConfig.java
##########
@@ -44,6 +44,13 @@
       .sinceVersion("0.7.0")
       .withDocumentation("Enable the internal metadata table which serves 
table metadata like level file listings");
 
+  // Enable syncing the Metadata Table
+  public static final ConfigProperty<Boolean> METADATA_SYNC_ENABLE_PROP = 
ConfigProperty

Review comment:
       this config and that config `hoodie.metadata.enable` confused me, from 
the users end, when setting `hoodie.metadata.enable` to true, we expect hudi 
should create and sync to metadata table, here we introduce another similar 
config and would make confusion.




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