Github user yuqi1129 commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5902#discussion_r183709218
  
    --- Diff: 
flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/history/HistoryServerArchiveFetcher.java
 ---
    @@ -211,7 +215,7 @@ public void run() {
                                                }
                                        }
                                        if (updateOverview) {
    -                                           updateJobOverview(webDir);
    +                                           
updateJobOverview(webOverviewDir, webDir);
                                        }
    --- End diff --
    
    In test case, it's quite clear this is make the HS has polled file before 
read, That is to say adding `numFinishedPolls` is just for test purpose, am i 
right ?


---

Reply via email to