[GitHub] spark issue #20799: [SPARK-23635][YARN] AM env variable should not overwrite...
Github user jerryshao commented on the issue: https://github.com/apache/spark/pull/20799 Thanks for the review, let me merge to master. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20799: [SPARK-23635][YARN] AM env variable should not overwrite...
Github user tgravescs commented on the issue: https://github.com/apache/spark/pull/20799 looks good to me --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20799: [SPARK-23635][YARN] AM env variable should not overwrite...
Github user mridulm commented on the issue: https://github.com/apache/spark/pull/20799 Thanks for the changes @jerryshao, they look good to me. Do you have any other thoughts/comments @tgravescs ? Since this is a behavior change, I want to make sure we are not missing anything ! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20799: [SPARK-23635][YARN] AM env variable should not overwrite...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20799 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88219/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20799: [SPARK-23635][YARN] AM env variable should not overwrite...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20799 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20799: [SPARK-23635][YARN] AM env variable should not overwrite...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20799 **[Test build #88219 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88219/testReport)** for PR 20799 at commit [`2afc91e`](https://github.com/apache/spark/commit/2afc91eef55bcbf2e79431d8db64d71fabc21585). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20799: [SPARK-23635][YARN] AM env variable should not overwrite...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20799 **[Test build #88219 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88219/testReport)** for PR 20799 at commit [`2afc91e`](https://github.com/apache/spark/commit/2afc91eef55bcbf2e79431d8db64d71fabc21585). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20799: [SPARK-23635][YARN] AM env variable should not overwrite...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20799 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20799: [SPARK-23635][YARN] AM env variable should not overwrite...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20799 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1498/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20799: [SPARK-23635][YARN] AM env variable should not overwrite...
Github user mridulm commented on the issue: https://github.com/apache/spark/pull/20799 Thanks for fixing this @jerryshao ! LGTM Since there is a behavior change, +CC @tgravescs and @vanzin --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20799: [SPARK-23635][YARN] AM env variable should not overwrite...
Github user jerryshao commented on the issue: https://github.com/apache/spark/pull/20799 @mridulm , would you please take a review. Thanks! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20799: [SPARK-23635][YARN] AM env variable should not overwrite...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20799 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20799: [SPARK-23635][YARN] AM env variable should not overwrite...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20799 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88168/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20799: [SPARK-23635][YARN] AM env variable should not overwrite...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20799 **[Test build #88168 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88168/testReport)** for PR 20799 at commit [`afd88e1`](https://github.com/apache/spark/commit/afd88e1a6585851f307413fd39a72167b9d8224c). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20799: [SPARK-23635][YARN] AM env variable should not overwrite...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20799 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1459/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20799: [SPARK-23635][YARN] AM env variable should not overwrite...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20799 **[Test build #88168 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88168/testReport)** for PR 20799 at commit [`afd88e1`](https://github.com/apache/spark/commit/afd88e1a6585851f307413fd39a72167b9d8224c). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20799: [SPARK-23635][YARN] AM env variable should not overwrite...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20799 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
