[ARROW-5844][Java] Remove if conditons in default float & double comp…

…arators
apache · Jul 10, 2019 · a921cdd · a921cdd
1 parent a4b4099
commit a921cdd
Showing 1 changed file with 2 additions and 16 deletions.
diff --git a/java/algorithm/src/main/java/org/apache/arrow/algorithm/sort/DefaultVectorComparators.java b/java/algorithm/src/main/java/org/apache/arrow/algorithm/sort/DefaultVectorComparators.java
@@ -132,14 +132,7 @@ public int compareNotNull(int index1, int index2) {
         }
       }
 
-      float result = Math.signum(value1 - value2);
-      if (result < 0f) {
-        return -1;
-      } else if (result > 0f) {
-        return 1;
-      } else {
-        return 0;
-      }
+      return (int) Math.signum(value1 - value2);
     }
   }
 
@@ -171,14 +164,7 @@ public int compareNotNull(int index1, int index2) {
         }
       }
 
-      double result = Math.signum(value1 - value2);
-      if (result < 0) {
-        return -1;
-      } else if (result > 0) {
-        return 1;
-      } else {
-        return 0;
-      }
+      return (int) Math.signum(value1 - value2);
     }
   }