Skip to content

Commit a2f65eb

Browse files
jiangxb1987HyukjinKwon
authored andcommitted
[MINOR][CORE][TEST] Remove unnecessary sort in UnsafeInMemorySorterSuite
## What changes were proposed in this pull request? We don't require specific ordering of the input data, the sort action is not necessary and misleading. ## How was this patch tested? Existing test suite. Author: Xingbo Jiang <[email protected]> Closes #21536 from jiangxb1987/sorterSuite. (cherry picked from commit 534065e) Signed-off-by: hyukjinkwon <[email protected]>
1 parent 470cacd commit a2f65eb

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

core/src/test/java/org/apache/spark/util/collection/unsafe/sort/UnsafeInMemorySorterSuite.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,6 @@ public int compare(
129129
final UnsafeSorterIterator iter = sorter.getSortedIterator();
130130
int iterLength = 0;
131131
long prevPrefix = -1;
132-
Arrays.sort(dataToSort);
133132
while (iter.hasNext()) {
134133
iter.loadNext();
135134
final String str =

0 commit comments

Comments
 (0)