Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20292
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20292#discussion_r163510314
--- Diff:
core/src/main/scala/org/apache/spark/util/collection/ExternalAppendOnlyMap.scala
---
@@ -528,14 +528,22 @@ class ExternalAppendOnlyMap[K, V,
Github user caneGuy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20292#discussion_r163160055
--- Diff:
core/src/main/scala/org/apache/spark/util/collection/ExternalAppendOnlyMap.scala
---
@@ -463,21 +463,21 @@ class ExternalAppendOnlyMap[K, V, C](
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20292#discussion_r163149829
--- Diff:
core/src/main/scala/org/apache/spark/util/collection/ExternalAppendOnlyMap.scala
---
@@ -547,9 +555,9 @@ class ExternalAppendOnlyMap[K, V,
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20292#discussion_r163149674
--- Diff:
core/src/main/scala/org/apache/spark/util/collection/ExternalAppendOnlyMap.scala
---
@@ -509,8 +509,8 @@ class ExternalAppendOnlyMap[K, V,
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20292#discussion_r163149478
--- Diff:
core/src/main/scala/org/apache/spark/util/collection/ExternalAppendOnlyMap.scala
---
@@ -463,21 +463,21 @@ class ExternalAppendOnlyMap[K, V,
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20292#discussion_r163149213
--- Diff:
core/src/main/scala/org/apache/spark/util/collection/ExternalAppendOnlyMap.scala
---
@@ -463,21 +463,21 @@ class ExternalAppendOnlyMap[K, V,
Github user caneGuy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20292#discussion_r163146577
--- Diff:
core/src/main/scala/org/apache/spark/util/collection/ExternalAppendOnlyMap.scala
---
@@ -463,21 +463,21 @@ class ExternalAppendOnlyMap[K, V, C](
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20292#discussion_r163074869
--- Diff:
core/src/main/scala/org/apache/spark/util/collection/ExternalAppendOnlyMap.scala
---
@@ -463,21 +463,21 @@ class ExternalAppendOnlyMap[K, V,
Github user caneGuy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20292#discussion_r162235896
--- Diff:
core/src/main/scala/org/apache/spark/util/collection/ExternalAppendOnlyMap.scala
---
@@ -509,8 +509,8 @@ class ExternalAppendOnlyMap[K, V, C](
Github user caneGuy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20292#discussion_r162235297
--- Diff:
core/src/main/scala/org/apache/spark/util/collection/ExternalAppendOnlyMap.scala
---
@@ -509,8 +509,8 @@ class ExternalAppendOnlyMap[K, V, C](
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20292#discussion_r162151603
--- Diff:
core/src/main/scala/org/apache/spark/util/collection/ExternalAppendOnlyMap.scala
---
@@ -509,8 +509,8 @@ class ExternalAppendOnlyMap[K, V,
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/20292#discussion_r162054765
--- Diff:
core/src/main/scala/org/apache/spark/util/collection/ExternalAppendOnlyMap.scala
---
@@ -547,9 +550,9 @@ class ExternalAppendOnlyMap[K, V, C](
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/20292#discussion_r162054687
--- Diff:
core/src/main/scala/org/apache/spark/util/collection/ExternalAppendOnlyMap.scala
---
@@ -463,21 +463,21 @@ class ExternalAppendOnlyMap[K, V, C](
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/20292#discussion_r162054413
--- Diff:
core/src/main/scala/org/apache/spark/util/collection/ExternalAppendOnlyMap.scala
---
@@ -463,21 +463,21 @@ class ExternalAppendOnlyMap[K, V, C](
GitHub user caneGuy opened a pull request:
https://github.com/apache/spark/pull/20292
[SPARK-23129][CORE] Make deserializeStream of DiskMapIterator init lazily
## What changes were proposed in this pull request?
Currently,the deserializeStream in
16 matches
Mail list logo