Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22203
Thanks! Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: r
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22203
Done, I added one example test failure to the description here.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user rednaxelafx commented on the issue:
https://github.com/apache/spark/pull/22203
Thanks @srowen ! In that case could we add a few mentions of such test
cases that failed into the PR description itself, perhaps a few lines under the
"How was this patch tested?" section to giv
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22203
Good point @rednaxelafx though I know some existing tests already
definitely failed without this change. They're in the JIRA. It's pervasive
enough that we'd see a number of failures due to this issu
Github user rednaxelafx commented on the issue:
https://github.com/apache/spark/pull/22203
I'm wondering, though, either in the same PR or in a separate PR, could we
add a test case in Spark that would trigger the "two non-abstract method" error
so that we'd show an example scenario i
Github user rednaxelafx commented on the issue:
https://github.com/apache/spark/pull/22203
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22203
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95159/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22203
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/22203
**[Test build #95159 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95159/testReport)**
for PR 22203 at commit
[`848bf1c`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22203
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/2488/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22203
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/22203
**[Test build #95159 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95159/testReport)**
for PR 22203 at commit
[`848bf1c`](https://github.com/apache/spark/commit/84
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22203
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/22203
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95158/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22203
**[Test build #95158 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95158/testReport)**
for PR 22203 at commit
[`b7c7bdb`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22203
**[Test build #95158 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95158/testReport)**
for PR 22203 at commit
[`b7c7bdb`](https://github.com/apache/spark/commit/b7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22203
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/2487/
17 matches
Mail list logo
