danny0405 commented on code in PR #11124:
URL: https://github.com/apache/hudi/pull/11124#discussion_r1587037847
##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/client/BaseHoodieTableServiceClient.java:
##########
@@ -798,7 +795,9 @@ protected void archive(HoodieTable table) {
}
try {
final Timer.Context timerContext = metrics.getArchiveCtx();
- // We cannot have unbounded commit files. Archive commits if we have to
archive
+ // We cannot have unbounded commit files. Archive commits if we have to
archive.
+ // Reload table timeline to reflect the latest commit.
+ table.getMetaClient().reloadActiveTimeline();
Review Comment:
Only the archive needs this now because the cleaning may be executed before
the archiving, and the archiving needs to see these cleaning commits. In all
other other cases, there is no need to refresh either the timeline or the fs
view.
--
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]
