Github user markhamstra commented on the issue:
https://github.com/apache/spark/pull/22039
> Yes, "quick hack", but, as opposed to what in these specific cases?
Yes, that is the key question. I'll admit, I haven't looked at all deeply
to try to figure out whether something bet
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22039
Yes, I agree about the future-proofing problem. However, at the moment, new
future cases will also not cause any compile problem, but will just also
trigger a MatchError. This is, simply, always a pr
Github user markhamstra commented on the issue:
https://github.com/apache/spark/pull/22039
Hmmm... sorry to be late to this, but making pattern matches exhaustive by
adding a catch-all case that then throws an exception, while easy, should be
considered as a less than optimal fix. Ide
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22039
Also merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22039
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94435/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22039
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/22039
**[Test build #94435 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94435/testReport)**
for PR 22039 at commit
[`e9220b4`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22039
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/22039
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94440/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22039
**[Test build #94440 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94440/testReport)**
for PR 22039 at commit
[`fc5d7a1`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22039
**[Test build #94440 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94440/testReport)**
for PR 22039 at commit
[`fc5d7a1`](https://github.com/apache/spark/commit/fc
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22039
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/22039
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-unified/1957/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22039
**[Test build #94435 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94435/testReport)**
for PR 22039 at commit
[`e9220b4`](https://github.com/apache/spark/commit/e9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22039
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-unified/1952/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22039
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22039
cc @kiszk @srowen @HyukjinKwon
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
17 matches
Mail list logo
