You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by zentol <gi...@git.apache.org> on 2018/05/07 09:44:59 UTC

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

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

    https://github.com/apache/flink/pull/5902#discussion_r186376796
  
    --- 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 --
    
    this change effectively kills backwards compatibility; i.e. a 1.4 HS cannot properly handle 1.5 archives, and a 1.5 HS cannot properly handle 1.4 archives. What is missing is a compatibility check for the old URL.


---