[ 
https://issues.apache.org/jira/browse/FLINK-9194?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16447973#comment-16447973
 ] 

ASF GitHub Bot commented on FLINK-9194:
---------------------------------------

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

    https://github.com/apache/flink/pull/5888#discussion_r183356809
  
    --- Diff: 
flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/history/HistoryServerArchiveFetcher.java
 ---
    @@ -231,18 +243,20 @@ public void run() {
         * <p>For the display in the HistoryServer WebFrontend we have to 
combine these overviews.
         */
        private static void updateJobOverview(File webDir) {
    -           File webOverviewDir = new File(webDir, "overviews");
    -           try (JsonGenerator gen = 
jacksonFactory.createGenerator(HistoryServer.createOrGetFile(webDir, 
"joboverview"))) {
    +           File webOverviewDir = new File(webDir, "/jobs/");
    +           try (JsonGenerator gen = 
jacksonFactory.createGenerator(HistoryServer.createOrGetFile(webDir, 
"/jobs/overview"))) {
                        gen.writeStartObject();
    -                   gen.writeArrayFieldStart("running");
    -                   gen.writeEndArray();
    -                   gen.writeArrayFieldStart("finished");
    +                   gen.writeArrayFieldStart("jobs");
     
                        File[] overviews = new 
File(webOverviewDir.getPath()).listFiles();
                        if (overviews != null) {
                                for (File overview : overviews) {
    -                                   JsonNode root = 
mapper.readTree(overview);
    -                                   JsonNode finished = 
root.get("finished");
    +                                   if (overview.isFile()) {
    +                                           continue;
    +                                   }
    +                                   File file = new 
File(overview.getAbsolutePath() + "/overview.json");
    +                                   JsonNode root = mapper.readTree(file);
    --- End diff --
    
    A much safer implementation would be to parse these into 
`MultipleJobsDetails`, merge them and write them back out.


> Finished jobs are not archived to HistoryServer
> -----------------------------------------------
>
>                 Key: FLINK-9194
>                 URL: https://issues.apache.org/jira/browse/FLINK-9194
>             Project: Flink
>          Issue Type: Bug
>          Components: History Server, JobManager
>    Affects Versions: 1.5.0
>         Environment: Flink 2af481a
>            Reporter: Gary Yao
>            Assignee: Chesnay Schepler
>            Priority: Blocker
>              Labels: flip-6
>             Fix For: 1.6.0
>
>
> In flip6 mode, jobs are not archived to the HistoryServer. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to