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

Reply via email to