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

    https://github.com/apache/flink/pull/1338#discussion_r45100162
  
    --- Diff: 
flink-core/src/main/java/org/apache/flink/configuration/ConfigConstants.java ---
    @@ -635,8 +644,18 @@
         * The default number of archived jobs for the jobmanager
         */
        public static final int DEFAULT_JOB_MANAGER_WEB_ARCHIVE_COUNT = 5;
    -   
    -   
    +
    +   /**
    +    * By default, submitting jobs from the web-frontend is allowed.
    +    */
    +   public static final boolean DEFAULT_JOB_MANAGER_WEB_SUBMISSION = true;
    +
    +   /**
    +    * Default directory for uploaded file storage for the Web frontend.
    +    */
    +   public static final String DEFAULT_JOB_MANAGER_WEB_UPLOAD_DIR =
    +                   (System.getProperty("java.io.tmpdir") == null ? "/tmp" 
: System.getProperty("java.io.tmpdir")) + "/webmonitor/";
    --- End diff --
    
    How about handling the temp directory similar to the IOManager and 
BlobManager: We create directory in the temporary directory (with a random 
name) and use a shutdown hook to remove the directory on shutdown again.
    
    I don't like keeping the jar files because it will pollute the directory on 
the server.
    Also the current approach does not allow running multiple JobManagers at 
the same machine (something that can easily happen with YARN).


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to