nsivabalan commented on code in PR #18353:
URL: https://github.com/apache/hudi/pull/18353#discussion_r2988703012
##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/metadata/HoodieBackedTableMetadataWriter.java:
##########
@@ -450,6 +450,13 @@ private boolean initializeFromFilesystem(String
dataTableInstantTime, List<Metad
}
}
+ // for a fresh table, lets defer RLI initialization
+ if (dataWriteConfig.getMetadataConfig().shouldDeferRliInitForFreshTable()
&& this.enabledPartitionTypes.contains(RECORD_INDEX)
+ &&
dataMetaClient.getActiveTimeline().filterCompletedInstants().countInstants() ==
0) {
+ this.enabledPartitionTypes.remove(RECORD_INDEX);
+ partitionsToInit.remove(RECORD_INDEX);
+ }
Review Comment:
actually, thats the reason I wanted to update `enabledPartitionTypes`. bcoz,
the incremental update code blocks could rely on `enabledPartitionTypes`
(https://github.com/apache/hudi/blob/9859f9aa29df124069c8d8d12df44a1635d48380/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/metadata/HoodieBackedTableMetadataWriter.java#L1273)
to determine the partitions to update. and if RLI is not bootstrapped here at
L457, we do not want the incremental commits to write to RLI.
--
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]
