Github user bersprockets commented on the issue: https://github.com/apache/spark/pull/22079 @jiangxb1987 Here are some of the differences from the original PR - I also ported the follow up PR #20426 - I ported #20088 (for SPARK-22905) to get the tests to pass. I also ported its followup, #20113. - sql/catalyst/src/main/java/org/apache/spark/sql/execution/UnsafeExternalRowSorter.java line 112 UnsafeExternalSorter.create does not take a Supplier for the 5th argument, so I put get() on the argument to directly pass a RecordComparator object. - sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/ShuffleExchange.scala: I access SQLConf differently (since SQLConf.get doesn't appear to work in 2.2).
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org