sandeep-katta commented on a change in pull request #25889:
[SPARK-29202][Deploy]driver java options are not passed to driver process in
Yarn client mode
URL: https://github.com/apache/spark/pull/25889#discussion_r326930875
##########
File path:
launcher/src/main/java/org/apache/spark/launcher/SparkSubmitCommandBuilder.java
##########
@@ -392,7 +392,7 @@ boolean isClientMode(Map<String, String> userProps) {
// Default master is "local[*]", so assume client mode in that case
return userMaster == null ||
"client".equals(userDeployMode) ||
- (!userMaster.equals("yarn") && userDeployMode == null);
+ userDeployMode == null;
Review comment:
ya ll add testcase and update the PR
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
