Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20841
Merged to master and branch-2.3.
Thank you @ueshin, @BryanCutler and @viirya for reviewing this.
---
-
To unsubscrib
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20841
LGTM.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apach
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20841
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/20841
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88339/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20841
**[Test build #88339 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88339/testReport)**
for PR 20841 at commit
[`3a8de04`](https://github.com/apache/spark/commit/3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20841
**[Test build #88339 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88339/testReport)**
for PR 20841 at commit
[`3a8de04`](https://github.com/apache/spark/commit/3a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20841
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/1581/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20841
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20841
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20841
**[Test build #88336 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88336/testReport)**
for PR 20841 at commit
[`3a8de04`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20841
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88336/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20841
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/20841
**[Test build #88336 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88336/testReport)**
for PR 20841 at commit
[`3a8de04`](https://github.com/apache/spark/commit/3a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20841
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/20841
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/1578/
Tes
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20841
Ah, I got the point. So, you mean a dictionary-like approach -
`get("non-existant-key")` should return `None` always instead of throwing an
error, language-specifically? I think we might conside
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/20841
Yeah, true that is how the Scala side works so I suppose that is best, but
I kind of view this api similar to the python `dict.get` which returns None and
won't raise an error, and that would ac
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20841
I think we should better match Python side behaviour to Scala side in
general and throw a better exception if possible. I think the application conf
was done like that because there wasn't `_NoV
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/20841
Shouldn't it be the same as in the application conf here
https://github.com/apache/spark/blob/master/python/pyspark/conf.py#L174?
Here the default is `None` and it returns `None` if th
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20841
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88300/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20841
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/20841
**[Test build #88300 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88300/testReport)**
for PR 20841 at commit
[`d8ee18f`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20841
**[Test build #88300 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88300/testReport)**
for PR 20841 at commit
[`d8ee18f`](https://github.com/apache/spark/commit/d8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20841
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/20841
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/1555/
Tes
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20841
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20841
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/20841
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88298/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20841
**[Test build #88298 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88298/testReport)**
for PR 20841 at commit
[`d8ee18f`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20841
**[Test build #88298 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88298/testReport)**
for PR 20841 at commit
[`d8ee18f`](https://github.com/apache/spark/commit/d8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20841
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/20841
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/1553/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20841
**[Test build #88294 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88294/testReport)**
for PR 20841 at commit
[`1a6cfce`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20841
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/20841
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88294/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20841
**[Test build #88294 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88294/testReport)**
for PR 20841 at commit
[`1a6cfce`](https://github.com/apache/spark/commit/1a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20841
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/1551/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20841
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20841
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/20841
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/20841
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88291/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20841
**[Test build #88291 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88291/testReport)**
for PR 20841 at commit
[`1a6cfce`](https://github.com/apache/spark/commit/1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20841
**[Test build #88291 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88291/testReport)**
for PR 20841 at commit
[`1a6cfce`](https://github.com/apache/spark/commit/1a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20841
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/1549/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20841
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20841
cc @cloud-fan, @viirya, @ueshin, @BryanCutler who I can directly think of
for now.
---
-
To unsubscribe, e-mail: reviews-uns
46 matches
Mail list logo