Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20080
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86090/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20080
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20080
**[Test build #86090 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86090/testReport)**
for PR 20080 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20080
**[Test build #86090 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86090/testReport)**
for PR 20080 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20080
Jenkins, retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20080
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86083/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20080
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20080
**[Test build #86083 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86083/testReport)**
for PR 20080 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20080
**[Test build #86083 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86083/testReport)**
for PR 20080 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20080
Ping @wangyum
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20080
This change alone is pretty harmless. If it seems to work as intended,
seems OK? Again, just wondering if `cachedExecutorIdleTimeoutS` should be the
same, and check for nonnegative values?
---
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/20080
@srowen @jiangxb1987 I have test on my cluster with this patch.
```
bin/spark-sql --master yarn --conf spark.dynamicAllocation.enabled=true
--conf spark.shuffle.service.enabled=true --conf
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20080
@wangyum any comments on the follow-up here? if you've observed it gets the
desired behavior, that's pretty convincing. At least we should treat both
timeouts consistently.
---
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20080
Good point @felixcheung though the polling is hard-coded to 100ms, so will
be close enough to "immediately". Allowing a value of 0 on both timeouts seems
OK, but yeah not clear whether it does have
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20080
@wangyum have you already run any jobs on a cluster with this patch?
---
-
To unsubscribe, e-mail:
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20080
I wonder if we should do the same for cachedExecutorIdleTimeoutS ? it isn't
checked for being nonnegative but should be, I suppose. I think this is fine
otherwise. I don't see any reason not to
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/20080
cc @srowen
---
-
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/20080
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85390/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20080
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20080
**[Test build #85390 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85390/testReport)**
for PR 20080 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20080
**[Test build #85390 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85390/testReport)**
for PR 20080 at commit
21 matches
Mail list logo