[flink] branch release-1.14 updated: [FLINK-24960][yarn-tests] change order of statements to avoid race condition

[mapohl]

This is an automated email from the ASF dual-hosted git repository.

mapohl pushed a commit to branch release-1.14
in repository https://gitbox.apache.org/repos/asf/flink.git


The following commit(s) were added to refs/heads/release-1.14 by this push:
     new 5275206fcb8 [FLINK-24960][yarn-tests] change order of statements to 
avoid race condition
5275206fcb8 is described below

commit 5275206fcb891d67153e108f2e81b3e2734b4784
Author: Niklas Semmler <nik...@ververica.com>
AuthorDate: Tue May 31 13:26:47 2022 +0200

    [FLINK-24960][yarn-tests] change order of statements to avoid race condition
---
 .../main/java/org/apache/flink/yarn/cli/FlinkYarnSessionCli.java    | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git 
a/flink-yarn/src/main/java/org/apache/flink/yarn/cli/FlinkYarnSessionCli.java 
b/flink-yarn/src/main/java/org/apache/flink/yarn/cli/FlinkYarnSessionCli.java
index b43dde42569..0c865eb3a40 100644
--- 
a/flink-yarn/src/main/java/org/apache/flink/yarn/cli/FlinkYarnSessionCli.java
+++ 
b/flink-yarn/src/main/java/org/apache/flink/yarn/cli/FlinkYarnSessionCli.java
@@ -611,10 +611,12 @@ public class FlinkYarnSessionCli extends AbstractYarnCli {
                     yarnApplicationId = clusterClient.getClusterId();
 
                     try {
+                        // Other threads use the Yarn properties file to 
connect to the YARN session
+                        writeYarnPropertiesFile(yarnApplicationId, 
dynamicPropertiesEncoded);
+
+                        // Multiple tests match on the following output
                         System.out.println(
                                 "JobManager Web Interface: " + 
clusterClient.getWebInterfaceURL());
-
-                        writeYarnPropertiesFile(yarnApplicationId, 
dynamicPropertiesEncoded);
                     } catch (Exception e) {
                         try {
                             clusterClient.close();