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

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_r183355554
  
    --- Diff: 
flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/history/HistoryServerArchiveFetcher.java
 ---
    @@ -180,10 +180,22 @@ public void run() {
     
                                                                        // We 
overwrite existing files since this may be another attempt at fetching this 
archive.
                                                                        // 
Existing files may be incomplete/corrupt.
    +
    +                                                                   //this 
is to save a job overview information, we put it in the ${path}/jobId folder
    +                                                                   
//because if put it in ${path}, new job inforamtion will override it and the web
    +                                                                   //will 
show only one job all the same.
    +                                                                   if 
(!target.toURI().toURL().getPath().contains(jobID)) {
    +                                                                           
Files.deleteIfExists(targetPath);
    +                                                                           
targetPath = new File(target.getParent() + "/" + jobID + "/" + 
target.getName()).toPath();
    +                                                                   }
                                                                        
Files.deleteIfExists(targetPath);
     
    -                                                                   
Files.createFile(target.toPath());
    -                                                                   try 
(FileWriter fw = new FileWriter(target)) {
    +                                                                   if 
(!targetPath.toFile().getParentFile().exists()) {
    --- End diff --
    
    this change is unnecessary if the above change is reverted, since 
directories are already created at Line 174.


> 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