jonvex commented on code in PR #13603:
URL: https://github.com/apache/hudi/pull/13603#discussion_r2242867516
##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/metadata/HoodieBackedTableMetadataWriter.java:
##########
@@ -1291,10 +1364,52 @@ protected Pair<List<HoodieFileGroupId>,
HoodieData<HoodieRecord>> tagRecordsWith
@Override
public void update(HoodieCommitMetadata commitMetadata, String instantTime) {
mayBeReinitMetadataReader();
+ maybeInitializeNewFileGroupsForPartitionedRLI(commitMetadata, instantTime);
processAndCommit(instantTime, new
BatchMetadataConversionFunction(instantTime, commitMetadata,
getMetadataPartitionsToUpdate()));
closeInternal();
}
+ /**
+ * This method is used to initialize new file groups for partitioned record
index during the in-flight commit.
+ * It will initialize new file groups for partitions that are newly added in
the inflight commit.
+ *
+ * @param commitMetadata metadata for the inflight commit
+ * @param instantTime Timestamp for the mdt commit
+ */
+ private void
maybeInitializeNewFileGroupsForPartitionedRLI(HoodieCommitMetadata
commitMetadata, String instantTime) {
Review Comment:
Because they call HoodieBackedTableMetadataWriter methods inside
--
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]
