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

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

asfgit closed pull request #6836: [FLINK-10516] [yarn] fix 
YarnApplicationMasterRunner fail to initiali…
URL: https://github.com/apache/flink/pull/6836
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git 
a/flink-yarn/src/main/java/org/apache/flink/yarn/YarnApplicationMasterRunner.java
 
b/flink-yarn/src/main/java/org/apache/flink/yarn/YarnApplicationMasterRunner.java
index e98e174cbdb..c275bfe0a02 100644
--- 
a/flink-yarn/src/main/java/org/apache/flink/yarn/YarnApplicationMasterRunner.java
+++ 
b/flink-yarn/src/main/java/org/apache/flink/yarn/YarnApplicationMasterRunner.java
@@ -27,6 +27,7 @@
 import org.apache.flink.configuration.ResourceManagerOptions;
 import org.apache.flink.configuration.SecurityOptions;
 import org.apache.flink.configuration.WebOptions;
+import org.apache.flink.core.fs.FileSystem;
 import org.apache.flink.runtime.akka.AkkaUtils;
 import org.apache.flink.runtime.clusterframework.BootstrapTools;
 import 
org.apache.flink.runtime.clusterframework.ContaineredTaskManagerParameters;
@@ -67,6 +68,7 @@
 import org.slf4j.LoggerFactory;
 
 import java.io.File;
+import java.io.IOException;
 import java.util.Map;
 import java.util.concurrent.Callable;
 import java.util.concurrent.ExecutorService;
@@ -163,6 +165,13 @@ protected int run(String[] args) {
 
                        final Configuration flinkConfig = 
createConfiguration(currDir, dynamicProperties, LOG);
 
+                       // configure the filesystems
+                       try {
+                               FileSystem.initialize(flinkConfig);
+                       } catch (IOException e) {
+                               throw new IOException("Error while configuring 
the filesystems.", e);
+                       }
+
                        File f = new File(currDir, Utils.KEYTAB_FILE_NAME);
                        if (remoteKeytabPrincipal != null && f.exists()) {
                                String keytabPath = f.getAbsolutePath();


 

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


> YarnApplicationMasterRunner fail to initialize FileSystem with correct Flink 
> Configuration during setup
> -------------------------------------------------------------------------------------------------------
>
>                 Key: FLINK-10516
>                 URL: https://issues.apache.org/jira/browse/FLINK-10516
>             Project: Flink
>          Issue Type: Bug
>          Components: YARN
>    Affects Versions: 1.4.0, 1.5.0, 1.6.0, 1.7.0
>            Reporter: Shuyi Chen
>            Assignee: Shuyi Chen
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.7.0, 1.6.2, 1.5.5
>
>
> Will add a fix, and refactor YarnApplicationMasterRunner to add a unittest to 
> prevent future regression.



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

Reply via email to