[GitHub] [spark] cxzl25 commented on issue #26323: [SPARK-29657][CORE] Iterator spill supporting radix sort with null prefix

2019-11-18 Thread GitBox
cxzl25 commented on issue #26323: [SPARK-29657][CORE] Iterator spill supporting radix sort with null prefix URL: https://github.com/apache/spark/pull/26323#issuecomment-554949809 @wangyum @HyukjinKwon @srowen Can you look at this pr? Thank you. This pr was raised for a while but was

[GitHub] [spark] cxzl25 commented on issue #26323: [SPARK-29657][CORE] Iterator spill supporting radix sort with null prefix

2019-11-06 Thread GitBox
cxzl25 commented on issue #26323: [SPARK-29657][CORE] Iterator spill supporting radix sort with null prefix URL: https://github.com/apache/spark/pull/26323#issuecomment-550733905 @cloud-fan Can you look at this question? Can really improve the success rate in some extreme scenarios. ---

[GitHub] [spark] cxzl25 commented on issue #26323: [SPARK-29657][CORE] Iterator spill supporting radix sort with null prefix

2019-11-02 Thread GitBox
cxzl25 commented on issue #26323: [SPARK-29657][CORE] Iterator spill supporting radix sort with null prefix URL: https://github.com/apache/spark/pull/26323#issuecomment-549031424 @JoshRosen @davies Can you look at this pr when you are free? Thank you. -

[GitHub] [spark] cxzl25 commented on issue #26323: [SPARK-29657][CORE] Iterator spill supporting radix sort with null prefix

2019-10-30 Thread GitBox
cxzl25 commented on issue #26323: [SPARK-29657][CORE] Iterator spill supporting radix sort with null prefix URL: https://github.com/apache/spark/pull/26323#issuecomment-547856822 ```UnsafeInMemorySorter#getSortedIterator``` Use radix sort, some keyPrefix has null, return ChainedIterato