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: commits-unsubscr...@hudi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org