[GitHub] spark pull request #23058: [SPARK-25905][CORE] When getting a remote block, ...

2018-11-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/23058 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request #23058: [SPARK-25905][CORE] When getting a remote block, ...

2018-11-28 Thread wypoon
Github user wypoon commented on a diff in the pull request: https://github.com/apache/spark/pull/23058#discussion_r237320541 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala --- @@ -718,13 +718,9 @@ private[spark] class BlockManager( }

[GitHub] spark pull request #23058: [SPARK-25905][CORE] When getting a remote block, ...

2018-11-28 Thread wypoon
Github user wypoon commented on a diff in the pull request: https://github.com/apache/spark/pull/23058#discussion_r237320424 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala --- @@ -789,21 +785,31 @@ private[spark] class BlockManager( }

[GitHub] spark pull request #23058: [SPARK-25905][CORE] When getting a remote block, ...

2018-11-28 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/23058#discussion_r237284719 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala --- @@ -718,13 +718,9 @@ private[spark] class BlockManager( }

[GitHub] spark pull request #23058: [SPARK-25905][CORE] When getting a remote block, ...

2018-11-28 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/23058#discussion_r237268983 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala --- @@ -789,21 +785,31 @@ private[spark] class BlockManager( }

[GitHub] spark pull request #23058: [SPARK-25905][CORE] When getting a remote block, ...

2018-11-21 Thread wypoon
Github user wypoon commented on a diff in the pull request: https://github.com/apache/spark/pull/23058#discussion_r235603798 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala --- @@ -693,9 +693,9 @@ private[spark] class BlockManager( */

[GitHub] spark pull request #23058: [SPARK-25905][CORE] When getting a remote block, ...

2018-11-20 Thread wypoon
Github user wypoon commented on a diff in the pull request: https://github.com/apache/spark/pull/23058#discussion_r235238066 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala --- @@ -693,9 +693,9 @@ private[spark] class BlockManager( */

[GitHub] spark pull request #23058: [SPARK-25905][CORE] When getting a remote block, ...

2018-11-19 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/23058#discussion_r234779890 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala --- @@ -693,9 +693,9 @@ private[spark] class BlockManager( */

[GitHub] spark pull request #23058: [SPARK-25905][CORE] When getting a remote block, ...

2018-11-19 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/23058#discussion_r234326643 --- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala --- @@ -693,9 +693,9 @@ private[spark] class BlockManager( */

[GitHub] spark pull request #23058: [SPARK-25905][CORE] When getting a remote block, ...

2018-11-16 Thread wypoon
GitHub user wypoon opened a pull request: https://github.com/apache/spark/pull/23058 [SPARK-25905][CORE] When getting a remote block, avoid forcing a conversion to a ChunkedByteBuffer ## What changes were proposed in this pull request? In `BlockManager`, `getRemoteValues`