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
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:
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
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
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
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/
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
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:
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
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
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
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
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/
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
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
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
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
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
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
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
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
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/
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:
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
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
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
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/
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
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
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
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
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/
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
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:
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
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
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
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/
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
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:
46 matches
Mail list logo
