nsivabalan commented on a change in pull request #3762:
URL: https://github.com/apache/hudi/pull/3762#discussion_r733654422
##########
File path:
hudi-common/src/main/java/org/apache/hudi/common/config/HoodieMetadataConfig.java
##########
@@ -115,6 +115,18 @@
.sinceVersion("0.7.0")
.withDocumentation("Parallelism to use, when listing the table on lake
storage.");
+ public static final ConfigProperty<Boolean> ENABLE_INLINE_READING_LOG_FILES
= ConfigProperty
+ .key(METADATA_PREFIX + ".enable.inline.reading.log.files")
+ .defaultValue(true)
+ .sinceVersion("0.10.0")
+ .withDocumentation("Enable inline reading of Log files");
+
+ public static final ConfigProperty<Boolean> ENABLE_FULL_SCAN_LOG_FILES =
ConfigProperty
+ .key(METADATA_PREFIX + ".enable.full.scan.log.files")
+ .defaultValue(true)
+ .sinceVersion("0.10.0")
+ .withDocumentation("Enable full scanning of log files while reading log
records");
Review comment:
wrt your suggestion of moving this out to common config(instead of
metadata config), I don't really see a need where we will use this for regular
data table. so, I prefer we can leave it at metadata config itself. let me know.
--
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]
