[GitHub] [flink] Myasuka edited a comment on issue #9007: [FLINK-13131] [Deployment / YARN] add a custom directory for storing configuration files and jar files for user jobs

[GitBox <gi...@apache.org>]

Myasuka edited a comment on issue #9007: [FLINK-13131] [Deployment / YARN] add a custom directory for storing configuration files and jar files for user jobs
URL: https://github.com/apache/flink/pull/9007#issuecomment-509013438
 
 
   @zhangbinzaifendou as you explained, the motivation to create this PR is due to your special online environment and your Hadoop. However, this is not the reason that you have to modify the Flink source code. I think the real solution is to modify your special Hadoop which caused this problem.
   
   As an open source software, Flink should adapt to the standard or widely-used ecosystems not some special customized ones. There would be no problems if we follow the correct logic of `FileSystem.getHomeDirectory()` even `dfs.user.home.dir.prefix` has been configured. And Flink use `FileSystem.getHomeDirectory()` to store its internal jars and files which is totally transparent to users. Exposing this internal implementation to users by adding such a configuration, especially user might configure this folder to a folder without permission, would never be a good idea.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services