Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22179
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/22179
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/2837/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22179
**[Test build #95673 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95673/testReport)**
for PR 22179 at commit
[`d1dac1e`](https://github.com/apache/spark/commit/d1
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22179
Thanks, @dongjoon-hyun
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22179
@wangyum
I made a [PR](https://github.com/wangyum/spark/pull/9) to your branch.
Could you review and merge that? The content are the followings.
- Add a new test case for SPARK-231
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22179
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95646/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22179
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/22179
**[Test build #95646 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95646/testReport)**
for PR 22179 at commit
[`0d78113`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22179
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95643/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22179
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/22179
**[Test build #95643 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95643/testReport)**
for PR 22179 at commit
[`f2fb28d`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22179
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/22179
**[Test build #95646 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95646/testReport)**
for PR 22179 at commit
[`0d78113`](https://github.com/apache/spark/commit/0d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22179
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/2813/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22179
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/2810/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22179
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/22179
**[Test build #95643 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95643/testReport)**
for PR 22179 at commit
[`f2fb28d`](https://github.com/apache/spark/commit/f2
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22179
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/22179
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95623/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22179
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/22179
**[Test build #95623 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95623/testReport)**
for PR 22179 at commit
[`f2fb28d`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22179
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/22179
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/2794/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22179
**[Test build #95623 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95623/testReport)**
for PR 22179 at commit
[`f2fb28d`](https://github.com/apache/spark/commit/f2
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22179
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/22179
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95613/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22179
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/22179
**[Test build #95613 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95613/testReport)**
for PR 22179 at commit
[`f2fb28d`](https://github.com/apache/spark/commit/f
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22179
Sorry @dongjoon-hyun I only reproduce one test.
kryo-parametrized-type-inheritance related to language. It seems scala
can't reproduce it:
```scala
val ser = new KryoSerializer(new
S
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22179
**[Test build #95613 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95613/testReport)**
for PR 22179 at commit
[`f2fb28d`](https://github.com/apache/spark/commit/f2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22179
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/22179
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/2787/
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22179
I pinged the first one, SPARK-20389, to find any chance to test on the
original cluster. Although it's too old issue, but there is no harm for us to
ping once more.
For the third one,
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22179
Tests are great of course; those three might be hard to test. The first and
last one don't have info on the reproduction. The second one looks possibly
reproducible though as it shows the custom seri
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22179
Thanks, @wangyum !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22179
@dongjoon-hyun I'm trying to add test cases.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional co
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22179
Ping, @wangyum . Could you add the test cases for SPARK-20389, SPARK-23131,
and SPARK-25176?
---
-
To unsubscribe, e-mail:
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22179
@wangyum . I know this is designed for a build-only PR. However, since this
is claimed for bug fixes(SPARK-20389, SPARK-23131, and SPARK-25176), could you
add test cases for them in order to v
Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/22179
For maximum breaking-change forgiveness, the "safe" option would be to punt
it to 3.0; which seems likely to be the next release after this.
---
-
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22179
Kryo 3.0.3 and Chill 0.8.2 already work with 2.12, so no issue particularly
there. I don't see that this change would be visible to users, and tests
suggest it doesn't affect Spark. I'm cautious that
Github user erikerlandson commented on the issue:
https://github.com/apache/spark/pull/22179
If it doesn't expose any breaking changes to users, that seems reasonable.
I assume there would be no integration problems with scala 2.12?
---
--
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22179
https://github.com/EsotericSoftware/kryo/releases/tag/kryo-parent-4.0.0
This has a number of breaking changes. However, Spark actually doesn't use
kryo itself, and excludes it. It uses kryo-s
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22179
Yes, I think the question is whether anything breaks as well. Kryo 4.x is a
breaking change IIRC. Is there a 3.x version that has these fixes?
---
--
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22179
Thanks @srowen
[SPARK-25176](https://issues.apache.org/jira/browse/SPARK-25176) has a detail
description:
> I'm using the latest spark version spark-core_2.11:2.3.1 which
transitively
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22179
That looks like a major version bump -- the usual question here -- what are
the key changes we need, what are possible incompatible changes?
---
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22179
cc @srowen
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22179
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95072/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22179
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/22179
**[Test build #95072 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95072/testReport)**
for PR 22179 at commit
[`8c28e07`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22179
**[Test build #95072 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95072/testReport)**
for PR 22179 at commit
[`8c28e07`](https://github.com/apache/spark/commit/8c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22179
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/2412/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22179
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
52 matches
Mail list logo