nsivabalan commented on code in PR #9367:
URL: https://github.com/apache/hudi/pull/9367#discussion_r1284886054
##########
hudi-common/src/main/java/org/apache/hudi/common/config/HoodieMetadataConfig.java:
##########
@@ -312,6 +312,13 @@ public final class HoodieMetadataConfig extends
HoodieConfig {
.withDocumentation("Maximum size in bytes of a single log file. Larger
log files can contain larger log blocks "
+ "thereby reducing the number of blocks to search for keys");
+ public static final ConfigProperty<Boolean> DISABLE_FILESYSTEM_BOOTSTRAP =
ConfigProperty
+ .key(METADATA_PREFIX + ".filesystem.bootstrap.disabled")
+ .defaultValue(false)
+ .sinceVersion("0.14.0")
+ .withDocumentation("Disable bootstrapping metadata table from the file
system when the table is first created. "
Review Comment:
should we consider explicitly calling external metadata sync in the config
naming.
current naming "hoodie.metadata.filesystem.bootstrap.disabled" is kind of
abstract. I am sure OSS dev may not understand what this config serves or when
would someone enable it.
Not too strong that we should change it. but just wanted to give it a 2nd
thought as the naming will stay for ever.
--
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]
