Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18711
LGTM, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18711
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18711
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82578/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18711
**[Test build #82578 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82578/testReport)**
for PR 18711 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18711
**[Test build #82578 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/82578/testReport)**
for PR 18711 at commit
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/18711
@10110346 Could you update the PR as @jerryshao suggested? Thanks!
---
-
To unsubscribe, e-mail:
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/18711
@JoshRosen @cloud-fan Could you help to review it ? thanks
---
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
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/18711
In the past days we often assume that in standalone mode, if we don't set
the "spark.executor.cores" property, then we should have no more than one
executor per host, but this PR explains a
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/18711
@srowen due to the current design of standalone cluster manager, if we
don't set `--total-executor-cores`, then Spark application will try to acquire
all the free cores on this cluster, and it
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18711
I don't object to this but I suppose I'm not clear if this was a point of
confusion that needed a fix.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18711
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80034/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18711
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18711
**[Test build #80034 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80034/testReport)**
for PR 18711 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18711
**[Test build #80034 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80034/testReport)**
for PR 18711 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18711
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79952/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18711
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18711
**[Test build #79952 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79952/testReport)**
for PR 18711 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18711
**[Test build #79952 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79952/testReport)**
for PR 18711 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18711
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79925/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18711
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18711
**[Test build #79925 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79925/testReport)**
for PR 18711 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18711
**[Test build #79925 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79925/testReport)**
for PR 18711 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18711
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79902/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18711
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18711
**[Test build #79902 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79902/testReport)**
for PR 18711 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18711
**[Test build #79902 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79902/testReport)**
for PR 18711 at commit
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/18711
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 feature
enabled and wishes so,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18711
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79898/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18711
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18711
**[Test build #79898 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79898/testReport)**
for PR 18711 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18711
**[Test build #79898 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79898/testReport)**
for PR 18711 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18711
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18711
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79895/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18711
**[Test build #79895 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79895/testReport)**
for PR 18711 at commit
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/18711
yea, I understand the issue now. Thank you for clarify!
---
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
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/18711
If the other application is finished, it will release cores
---
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
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/18711
How can we wait for new free cores on the same worker, if we only start one
executor for each worker initially?
---
If your project is set up for it, you can reply to this email and have your
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18711
**[Test build #79895 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79895/testReport)**
for PR 18711 at commit
Github user 10110346 commented on the issue:
https://github.com/apache/spark/pull/18711
@jiangxb1987 If `app.coresLeft` is not zero and there is no more free
cores left, it is not ending.Waiting for some workers have free cores, this
app will be assigned cores continue, and this
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/18711
AFAIK when `spark.executor.cores` is not set, we would keep assigning free
cores to the executor until `app.coresLeft` is zero or there is no more free
cores left. So I'm not convinced the
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18711
I'm not sure this is an improvement, though not really against it.
---
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 issue:
https://github.com/apache/spark/pull/18711
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79865/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18711
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18711
**[Test build #79865 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79865/testReport)**
for PR 18711 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18711
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18711
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79861/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18711
**[Test build #79861 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79861/testReport)**
for PR 18711 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18711
**[Test build #79865 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79865/testReport)**
for PR 18711 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18711
**[Test build #79861 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79861/testReport)**
for PR 18711 at commit
49 matches
Mail list logo