[GitHub] flink pull request #5902: [FLINK-9194][history] Add HistoryServer support to...

[yuqi1129]

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

    https://github.com/apache/flink/pull/5902#discussion_r186599431
  
    --- Diff: 
flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/history/HistoryServerArchiveFetcher.java
 ---
    @@ -162,7 +166,7 @@ public void run() {
                                                                        String 
json = archive.getJson();
     
                                                                        File 
target;
    -                                                                   if 
(path.equals("/joboverview")) {
    --- End diff --
    
    Agree


---