[GitHub] spark pull request #22404: DO NOT MERGE

2018-09-13 Thread squito
Github user squito closed the pull request at: https://github.com/apache/spark/pull/22404 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request #22404: DO NOT MERGE

2018-09-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/22404#discussion_r217154410 --- Diff: core/src/main/scala/org/apache/spark/api/python/PythonRDD.scala --- @@ -692,5 +681,238 @@ private[spark] class PythonBroadcast(@transient var

[GitHub] spark pull request #22404: DO NOT MERGE

2018-09-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/22404#discussion_r217154988 --- Diff: core/src/main/scala/org/apache/spark/api/python/PythonRunner.scala --- @@ -193,19 +193,51 @@ private[spark] abstract class BasePythonRunner[IN,

[GitHub] spark pull request #22404: DO NOT MERGE

2018-09-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/22404#discussion_r217153676 --- Diff: common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/SimpleDownloadFile.java --- @@ -0,0 +1,89 @@ +/* + * Licensed to

[GitHub] spark pull request #22404: DO NOT MERGE

2018-09-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/22404#discussion_r217153478 --- Diff: common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/SimpleDownloadFile.java --- @@ -0,0 +1,89 @@ +/* + * Licensed to

[GitHub] spark pull request #22404: DO NOT MERGE

2018-09-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/22404#discussion_r217153096 --- Diff: common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/DownloadFile.java --- @@ -0,0 +1,45 @@ +/* + * Licensed to the

[GitHub] spark pull request #22404: DO NOT MERGE

2018-09-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/22404#discussion_r217153301 --- Diff: common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/DownloadFile.java --- @@ -0,0 +1,45 @@ +/* + * Licensed to the

[GitHub] spark pull request #22404: DO NOT MERGE

2018-09-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/22404#discussion_r217153554 --- Diff: common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/SimpleDownloadFile.java --- @@ -0,0 +1,89 @@ +/* + * Licensed to

[GitHub] spark pull request #22404: DO NOT MERGE

2018-09-12 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/22404#discussion_r217155746 --- Diff: python/pyspark/context.py --- @@ -499,19 +506,32 @@ def f(split, iterator): def _serialize_to_jvm(self, data, parallelism,

[GitHub] spark pull request #22404: DO NOT MERGE

2018-09-12 Thread squito
GitHub user squito opened a pull request: https://github.com/apache/spark/pull/22404 DO NOT MERGE just for testing You can merge this pull request into a Git repository by running: $ git pull https://github.com/squito/spark assorted_2.3_fixes Alternatively you can review and