Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8910
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enab
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143745833
LGTM, merging to master and branch-1.5.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project do
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143691235
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143691233
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143691131
[Test build #43056 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43056/console)
for PR 8910 at commit
[`0e54fb1`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143656095
[Test build #43056 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43056/consoleFull)
for PR 8910 at commit
[`0e54fb1`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143655787
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143655773
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user jerryshao commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143655711
Jenkins, retest this please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143547947
[Test build #1818 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1818/console)
for PR 8910 at commit
[`0e54fb1`](https://github
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143547625
[Test build #1818 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1818/consoleFull)
for PR 8910 at commit
[`0e54fb1`](https://git
Github user jerryshao commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143387790
Looks like some other patches introduce this mima failures.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143385173
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143385172
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143385108
[Test build #43042 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43042/console)
for PR 8910 at commit
[`0e54fb1`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143383946
[Test build #43042 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43042/consoleFull)
for PR 8910 at commit
[`0e54fb1`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143383873
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143383860
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user jerryshao commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143383838
Jenkins, retest this please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143354587
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143354540
[Test build #43033 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43033/console)
for PR 8910 at commit
[`0e54fb1`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143354584
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143349972
[Test build #43033 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43033/consoleFull)
for PR 8910 at commit
[`0e54fb1`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143349374
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143349393
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143344733
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143344646
[Test build #43032 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43032/console)
for PR 8910 at commit
[`d53d5c8`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143344736
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143335895
[Test build #43032 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43032/consoleFull)
for PR 8910 at commit
[`d53d5c8`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143334652
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143334636
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8910#discussion_r40468010
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtil.scala ---
@@ -39,7 +39,7 @@ import org.apache.hadoop.yarn.util.ConverterUtils
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-14675
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feat
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143331462
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143331463
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143331411
[Test build #43030 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43030/console)
for PR 8910 at commit
[`d53d5c8`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143331198
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143331196
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143331056
[Test build #43027 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43027/console)
for PR 8910 at commit
[`e6547fb`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143324304
[Test build #43030 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43030/consoleFull)
for PR 8910 at commit
[`d53d5c8`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143323932
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143323853
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143304877
LGTM pending a couple of minor changes.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project do
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8910#discussion_r40457129
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtil.scala ---
@@ -314,5 +314,28 @@ object YarnSparkHadoopUtil {
def getClassP
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8910#discussion_r40456792
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtil.scala ---
@@ -314,5 +314,28 @@ object YarnSparkHadoopUtil {
def getClassP
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143289715
[Test build #43027 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43027/consoleFull)
for PR 8910 at commit
[`e6547fb`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143284017
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143284102
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user jerryshao commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143283745
Thanks @srowen for your review, I've updated the codes according to your
comments.
---
If your project is set up for it, you can reply to this email and have your
rep
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/8910#discussion_r40406320
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtil.scala ---
@@ -314,5 +314,28 @@ object YarnSparkHadoopUtil {
def getClassP
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/8910#discussion_r40406280
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtil.scala ---
@@ -314,5 +314,28 @@ object YarnSparkHadoopUtil {
def getClassP
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/8910#discussion_r40406303
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtil.scala ---
@@ -314,5 +314,28 @@ object YarnSparkHadoopUtil {
def getClassP
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143097960
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143097962
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143097899
[Test build #42987 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42987/console)
for PR 8910 at commit
[`c912a2a`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143070785
[Test build #42987 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42987/consoleFull)
for PR 8910 at commit
[`c912a2a`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143069576
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8910#issuecomment-143069548
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
GitHub user jerryshao opened a pull request:
https://github.com/apache/spark/pull/8910
[SPARK-10790][YARN] Fix initial executor number not set issue and
consolidate the codes
This bug is introduced in
[SPARK-9092](https://issues.apache.org/jira/browse/SPARK-9092),
`targetExecutorN
59 matches
Mail list logo