Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/21007
@gatorsmile I think the PR description here is great and very detailed,
what exactly is wrong and scary?
---
-
To unsubscrib
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21007
What's wrong in the description and PR title, and what to document? Do you
mean the first sentence `This PR proposes to add collect to a query executor as
an action.` is wrong because this sente
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21007
@HyukjinKwon @BryanCutler @viirya @felixcheung The first sentence of this
PR really scares me. After reading the PR description. Since the PR description
will be part of our change log. Please be
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21007
Merged to master.
Thanks for reviewing this @felixcheung, @viirya and @BryanCutler.
---
-
To unsubscribe, e-mail: re
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
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/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89304/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89304 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89304/testReport)**
for PR 21007 at commit
[`7c1b3c6`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2289/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89304 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89304/testReport)**
for PR 21007 at commit
[`7c1b3c6`](https://github.com/apache/spark/commit/7c
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21007
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89290/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89290 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89290/testReport)**
for PR 21007 at commit
[`7c1b3c6`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2285/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
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/21007
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/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2283/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89290 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89290/testReport)**
for PR 21007 at commit
[`7c1b3c6`](https://github.com/apache/spark/commit/7c
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/21007
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Merged build finished. Test FAILed.
---
-
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/21007
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89279/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89279 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89279/testReport)**
for PR 21007 at commit
[`7c1b3c6`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2275/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89279 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89279/testReport)**
for PR 21007 at commit
[`7c1b3c6`](https://github.com/apache/spark/commit/7c
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21007
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89267/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89267 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89267/testReport)**
for PR 21007 at commit
[`7c1b3c6`](https://github.com/apache/spark/commit/7
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21007
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2265/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89267 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89267/testReport)**
for PR 21007 at commit
[`7c1b3c6`](https://github.com/apache/spark/commit/7c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89193/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89193 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89193/testReport)**
for PR 21007 at commit
[`1a52dbe`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2208/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89193 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89193/testReport)**
for PR 21007 at commit
[`1a52dbe`](https://github.com/apache/spark/commit/1a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
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/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89182/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89182 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89182/testReport)**
for PR 21007 at commit
[`deacb17`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89182 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89182/testReport)**
for PR 21007 at commit
[`deacb17`](https://github.com/apache/spark/commit/de
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
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/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2200/
Tes
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21007
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Merged build finished. Test FAILed.
---
-
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/21007
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89173/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89173 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89173/testReport)**
for PR 21007 at commit
[`deacb17`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
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/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2193/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89173 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89173/testReport)**
for PR 21007 at commit
[`deacb17`](https://github.com/apache/spark/commit/de
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21007
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Merged build finished. Test FAILed.
---
-
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/21007
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89163/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89163 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89163/testReport)**
for PR 21007 at commit
[`deacb17`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
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/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2184/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89163 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89163/testReport)**
for PR 21007 at commit
[`deacb17`](https://github.com/apache/spark/commit/de
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21007
will address the comments soon.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
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/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89127/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89127 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89127/testReport)**
for PR 21007 at commit
[`e865c88`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89108/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89108 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89108/testReport)**
for PR 21007 at commit
[`edb5eea`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
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/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2156/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89127 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89127/testReport)**
for PR 21007 at commit
[`e865c88`](https://github.com/apache/spark/commit/e8
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21007
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89109/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89109 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89109/testReport)**
for PR 21007 at commit
[`e865c88`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
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/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2140/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89109 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89109/testReport)**
for PR 21007 at commit
[`e865c88`](https://github.com/apache/spark/commit/e8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
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/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2139/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89108 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89108/testReport)**
for PR 21007 at commit
[`edb5eea`](https://github.com/apache/spark/commit/ed
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21007
Will address the comments together soon.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional co
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21007
Will give a shot first for the Python one to show how it looks like. I have
an incomplete one in my local.
---
-
To unsubscr
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21007
got it. sorry I missed the last sentence. maybe a jvm only test?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apac
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21007
Yup, will add. I was just hesitant because It needs some complexity for
testing out (as described in the PR description) whereas the fix could be quite
straightforward.
---
--
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21007
@BryanCutler @ueshin
add test in
https://github.com/apache/spark/blob/master/python/pyspark/tests.py?
---
-
To unsubscr
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21007
cc @cloud-fan and @viirya (from checking the history).
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
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/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89053/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89053 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89053/testReport)**
for PR 21007 at commit
[`db1987f`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21007
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/21007
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2093/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21007
**[Test build #89053 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89053/testReport)**
for PR 21007 at commit
[`db1987f`](https://github.com/apache/spark/commit/db
92 matches
Mail list logo