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

Reply via email to