codope commented on code in PR #11634:
URL: https://github.com/apache/hudi/pull/11634#discussion_r1743755468
##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/metadata/HoodieBackedTableMetadataWriter.java:
##########
@@ -448,9 +448,9 @@ private boolean initializeFromFilesystem(String
initializationTime, List<Metadat
// Perform the commit using bulkCommit
HoodieData<HoodieRecord> records =
fileGroupCountAndRecordsPair.getValue();
- bulkCommit(commitTimeForPartition, partitionType, records,
fileGroupCount);
- metadataMetaClient.reloadActiveTimeline();
String partitionPath = (partitionType == FUNCTIONAL_INDEX ||
partitionType == SECONDARY_INDEX) ?
dataWriteConfig.getIndexingConfig().getIndexName() :
partitionType.getPartitionPath();
+ bulkCommit(commitTimeForPartition, partitionPath, records,
fileGroupCount);
Review Comment:
Have done some of the refactoring. Discussed offline with Ethan - we need a
separate abstraction, enum is not sufficient to pack everything. I will do that
in a subsequent PR.
--
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]
