Github user srowen commented on the issue:
https://github.com/apache/spark/pull/19846
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mpetruska commented on the issue:
https://github.com/apache/spark/pull/19846
@HyukjinKwon : Hi, yes: this is the "retrofitting" of the original bug fix,
which never made it to Scala 2.11. Instead of copy-pasting the original
discussion, let me link the [original
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/19846
@mpetruska, btw can you fill up the PR description about how it fixes?
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19846
**[Test build #3998 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3998/testReport)**
for PR 19846 at commit
Github user mpetruska commented on the issue:
https://github.com/apache/spark/pull/19846
@srowen Yes, exactly. Tried to minimize the risks of any problems caused by
this change.
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19846
**[Test build #3998 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3998/testReport)**
for PR 19846 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/19846
OK, so this just affects the Scala 2.11 REPL in Spark (good, because the
2.12 version works a little differently and the fix is already in 2.12), and
you've just copied in the fixed classes and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19846
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mpetruska commented on the issue:
https://github.com/apache/spark/pull/19846
Sorry, could not refactor the code better; since it makes heavy use of
path-dependent types.
---
-
To unsubscribe, e-mail:
