Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18277
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/18277
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86450/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18277
**[Test build #86450 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86450/testReport)**
for PR 18277 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18277
**[Test build #86450 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86450/testReport)**
for PR 18277 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18277
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18277
Let me merge this one only into master considering the concerns -
https://github.com/apache/spark/pull/18277#pullrequestreview-90007120 and
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18277
Let me merge this one in few days if there's no more comments.
---
-
To unsubscribe, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18277
cc @ueshin too. I think we were in several PRs related with encoding /
decoding stuff.
---
-
To unsubscribe, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18277
So, after this change, we will get rid of system default roundtrip in
**When `obj`: `unicode`** and **When `obj`: other types**.
In case of **When `obj`: other types**, we _might_ have
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18277
Wanted to make a clarification on what we will change here to myself
because it's quite confusing to me.
In Python 3, it's declared above `basestring = unicode = str`. So, it won't
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18277
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/18277
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86375/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18277
**[Test build #86375 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86375/testReport)**
for PR 18277 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18277
This change looks reasonable to me for now. But I'm also concerned about
the behavior change. A note into release notes should be good or maybe we need
a note at migration guide in `RDD Programming
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18277
**[Test build #86375 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86375/testReport)**
for PR 18277 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18277
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/18277
Jenkins OK to test.
---
-
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/18277
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18277
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85574/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18277
**[Test build #85574 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85574/testReport)**
for PR 18277 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18277
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/18277
**[Test build #85574 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85574/testReport)**
for PR 18277 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18277
ok to test
---
-
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/18277
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18277
it seems okay without a close look. Let me take the close look if I can
take the look first soon.
---
-
To unsubscribe,
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/18277
What do you think @HyukjinKwon ? I think this is probably a reasonable fix,
but we might break some peoples code who have been depending on the bug.
---
Github user sasameti commented on the issue:
https://github.com/apache/spark/pull/18277
how do I apply the patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user chaoslawful commented on the issue:
https://github.com/apache/spark/pull/18277
Well, the difference comes from repr()'s divergent default behaviors
between Python2 and Python3. And the previous code does no better than the
patched one but causing troubles while processing
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18277
When you try to do this on a rdd of array of unicode string. The result of
Python2 looks a bit weird.
Using Python version 2.7.12 (default, Jul 1 2016 15:12:24)
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18277
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
30 matches
Mail list logo
