Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/23228#discussion_r240036698
--- Diff:
core/src/main/scala/org/apache/spark/shuffle/sort/SortShuffleManager.scala ---
@@ -33,10 +33,10 @@ import org.apache.spark.shuffle._
*
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/23228#discussion_r239215561
--- Diff:
core/src/main/scala/org/apache/spark/shuffle/sort/SortShuffleManager.scala ---
@@ -33,10 +33,10 @@ import org.apache.spark.shuffle._
*
GitHub user 10110346 opened a pull request:
https://github.com/apache/spark/pull/23228
[MINOR][DOC]The condition description of serialized shuffle is not very
accurate
## What changes were proposed in this pull request?
`1. The shuffle dependency specifies no aggregation or
