[GitHub] [gobblin] abhisheknath2011 commented on a change in pull request #3349: Avro 1.9 upgrade compatible change - replaced deprecated public APIs with the compatible APIs

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

abhisheknath2011 commented on a change in pull request #3349:
URL: https://github.com/apache/gobblin/pull/3349#discussion_r686865360



##########
File path: gobblin-yarn/src/main/java/org/apache/gobblin/yarn/GobblinYarnAppLauncher.java
##########
@@ -253,8 +259,16 @@ public GobblinYarnAppLauncher(Config config, YarnConfiguration yarnConfiguration
     YarnHelixUtils.setYarnClassPath(config, this.yarnConfiguration);
     YarnHelixUtils.setAdditionalYarnClassPath(config, this.yarnConfiguration);
     this.yarnConfiguration.set("fs.automatic.close", "false");
-    this.yarnClient = YarnClient.createYarnClient();
-    this.yarnClient.init(this.yarnConfiguration);
+    this.originalYarnRMAddress = this.yarnConfiguration.get(GobblinYarnConfigurationKeys.YARN_RESOURCE_MANAGER_ADDRESS);

Review comment:
       Please ignore this as this came from rebase. This change was introduced by one of the first three commits in the master branch. These three commits are not available in avro_1_9 and got added after rebase from upstream master.




-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@gobblin.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org