zhangyue19921010 commented on a change in pull request #5186:
URL: https://github.com/apache/hudi/pull/5186#discussion_r839721790
##########
File path:
hudi-common/src/main/java/org/apache/hudi/common/table/HoodieTableConfig.java
##########
@@ -208,6 +209,11 @@
.sinceVersion("0.11.0")
.withDocumentation("Table checksum is used to guard against partial
writes in HDFS. It is added as the last entry in hoodie.properties and then
used to validate while reading table config.");
+ public static final ConfigProperty<Boolean> METADATA_TABLE_ENABLE =
ConfigProperty
+ .key("hoodie.table.metadata.enable")
+ .defaultValue(HoodieMetadataConfig.ENABLE.defaultValue())
+ .withDocumentation("Enable the internal metadata table which serves
table metadata like level file listings.");
Review comment:
Sure, we need to let users know it. Just thinking maybe we can let
reader side to notice the config in hoodie.properties:
1. if `METADATA_TABLE_ENABLE` false in hoodie.properties, no matter users
set on reader, we will use FileSystemBased
2. if `METADATA_TABLE_ENABLE` true in hoodie.properties, and users set MDT
true, then we use MDT Based action?
--
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]
