Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21209
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user Ngone51 commented on the issue:
https://github.com/apache/spark/pull/21209
ping @squito @vanzin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21209
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user Ngone51 commented on the issue:
https://github.com/apache/spark/pull/21209
ping @jiangxb1987
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/21209
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21209
**[Test build #90343 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90343/testReport)**
for PR 21209 at commit
[`264f316`](https://github.com/apache/spark/commit/26
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21209
**[Test build #90343 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90343/testReport)**
for PR 21209 at commit
[`264f316`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21209
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21209
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90343/
Test PASSed.
---
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/21209
LGTM. I have a feeling the current code would heal itself later (by the
fact that `ExecutorAllocationManager` calls `requestTotalExecutors`) but this
looks more correct.
Merging to master.
Github user Ngone51 commented on the issue:
https://github.com/apache/spark/pull/21209
Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.ap
11 matches
Mail list logo