Github user markhamstra commented on the issue:
https://github.com/apache/spark/pull/22176
Yes, this is better than what we had, but maybe it can be better still.
---
-
To unsubscribe, e-mail:
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/22176
@markhamstra That's a good point. However, since this is just following
our current codes if you check the usages of `newDaemonCachedThreadPool`, and
the changes here should be safe considering
Github user markhamstra commented on the issue:
https://github.com/apache/spark/pull/22176
@zsxwing we really should have considered whether this should be a
configuration variable instead of a fixed number of threads in any environment.
---
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/22176
LGTM. Merging to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mukulmurthy commented on the issue:
https://github.com/apache/spark/pull/22176
@zsxwing , @JoshRosen , @marmbrus for review and merge.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22176
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95066/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22176
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/22176
**[Test build #95066 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95066/testReport)**
for PR 22176 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22176
**[Test build #95066 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95066/testReport)**
for PR 22176 at commit
Github user mukulmurthy commented on the issue:
https://github.com/apache/spark/pull/22176
Jenkins, retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user mukulmurthy commented on the issue:
https://github.com/apache/spark/pull/22176
retest this please
---
-
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/22176
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95057/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22176
**[Test build #95057 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95057/testReport)**
for PR 22176 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22176
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mukulmurthy commented on the issue:
https://github.com/apache/spark/pull/22176
@JoshRosen @zsxwing @marmbrus to merge.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22176
**[Test build #95057 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95057/testReport)**
for PR 22176 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22176
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22176
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
18 matches
Mail list logo
