Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21859#discussion_r208775191
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/ShuffleExchangeExec.scala
---
@@ -294,7 +296,12 @@ object ShuffleExchangeExec {
sorter.sort(iter.asInstanceOf[Iterator[UnsafeRow]])
}
} else {
- rdd
+ part match {
+ case partitioner: RangePartitioner[InternalRow @unchecked, _]
--- End diff --
Do we need to match type argument of `RangePartitioner`?
Maybe just:
```scala
part match {
case partitioner: RangePartitioner[_, _] if partitioner.getSampledArray
!= null =>
...
case _ => rdd
}
```
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]