manojpec commented on a change in pull request #4067:
URL: https://github.com/apache/hudi/pull/4067#discussion_r764449725
##########
File path:
hudi-common/src/main/java/org/apache/hudi/common/table/log/HoodieMergedLogRecordScanner.java
##########
@@ -282,12 +286,35 @@ public Builder withPartition(String partitionName) {
return this;
}
+ public Builder enableFullScan(boolean enableFullScan) {
+ this.enableFullScan = enableFullScan;
+ return this;
+ }
+
@Override
public HoodieMergedLogRecordScanner build() {
+ if (HoodieTableMetadata.isMetadataTable(basePath)) {
Review comment:
HoodieCompactor has a config to read the log blocks lazily when merging
the records and to reverse read the logs. We don't expose these for other
users. So, the compactor directly used the base merged log record scanner to be
able to control this property. To suffice compactor needs we need this
detection and using the right record reader.
--
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]
