[SPARK-36047][CORE] Replace the handwriting compare methods with stat…

…ic compare methods in Java code

### What changes were proposed in this pull request?
The main change of this is use the static `Integer.compare()` method and `Long.compare()` method instead of the handwriting compare method in Java code.

### Why are the changes needed?
Removing unnecessary handwriting compare methods

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
Pass the Jenkins or GitHub Action

Closes apache#33260 from LuciferYang/static-compare.

Authored-by: yangjie01 <[email protected]>
Signed-off-by: Sean Owen <[email protected]>

core/src/main/java/org/apache/spark/shuffle/sort/ShuffleInMemorySorter.java

@@ -31,9 +31,7 @@ final class ShuffleInMemorySorter {
   private static final class SortComparator implements Comparator<PackedRecordPointer> {
     @Override
     public int compare(PackedRecordPointer left, PackedRecordPointer right) {
-      int leftId = left.getPartitionId();
-      int rightId = right.getPartitionId();
-      return leftId < rightId ? -1 : (leftId > rightId ? 1 : 0);
+      return Integer.compare(left.getPartitionId(), right.getPartitionId());
     }
   }
   private static final SortComparator SORT_COMPARATOR = new SortComparator();

core/src/main/java/org/apache/spark/util/collection/unsafe/sort/PrefixComparators.java

@@ -141,7 +141,7 @@ public static final class SignedPrefixComparator extends RadixSortSupport {
     @Override public boolean sortSigned() { return true; }
     @Override public boolean nullsFirst() { return true; }
     public int compare(long a, long b) {
-      return (a < b) ? -1 : (a > b) ? 1 : 0;
+      return Long.compare(a, b);
     }
   }
 
@@ -150,7 +150,7 @@ public static final class SignedPrefixComparatorNullsLast extends RadixSortSuppo
     @Override public boolean sortSigned() { return true; }
     @Override public boolean nullsFirst() { return false; }
     public int compare(long a, long b) {
-      return (a < b) ? -1 : (a > b) ? 1 : 0;
+      return Long.compare(a, b);
     }
   }
 
@@ -159,7 +159,7 @@ public static final class SignedPrefixComparatorDescNullsFirst extends RadixSort
     @Override public boolean sortSigned() { return true; }
     @Override public boolean nullsFirst() { return true; }
     public int compare(long b, long a) {
-      return (a < b) ? -1 : (a > b) ? 1 : 0;
+      return Long.compare(a, b);
     }
   }
 
@@ -168,7 +168,7 @@ public static final class SignedPrefixComparatorDesc extends RadixSortSupport {
     @Override public boolean sortSigned() { return true; }
     @Override public boolean nullsFirst() { return false; }
     public int compare(long b, long a) {
-      return (a < b) ? -1 : (a > b) ? 1 : 0;
+      return Long.compare(a, b);
     }
   }
 }