diff --git a/src/main/java/org/elasticsearch/index/fielddata/fieldcomparator/DoubleValuesComparatorBase.java b/src/main/java/org/elasticsearch/index/fielddata/fieldcomparator/DoubleValuesComparatorBase.java index bcaa047cfa32e..4af80a427ec10 100644 --- a/src/main/java/org/elasticsearch/index/fielddata/fieldcomparator/DoubleValuesComparatorBase.java +++ b/src/main/java/org/elasticsearch/index/fielddata/fieldcomparator/DoubleValuesComparatorBase.java @@ -62,13 +62,7 @@ public final FieldComparator setNextReader(AtomicReaderContext context) throw } static final int compare(double left, double right) { - if (left > right) { - return 1; - } else if (left < right) { - return -1; - } else { - return 0; - } + return Double.compare(left, right); } static final class MultiValueWrapper extends DoubleValues.Filtered { diff --git a/src/main/java/org/elasticsearch/index/fielddata/fieldcomparator/FloatValuesComparator.java b/src/main/java/org/elasticsearch/index/fielddata/fieldcomparator/FloatValuesComparator.java index 960c60b0ca777..0a41d9b3f1665 100644 --- a/src/main/java/org/elasticsearch/index/fielddata/fieldcomparator/FloatValuesComparator.java +++ b/src/main/java/org/elasticsearch/index/fielddata/fieldcomparator/FloatValuesComparator.java @@ -39,13 +39,7 @@ public FloatValuesComparator(IndexNumericFieldData indexFieldData, float miss public int compare(int slot1, int slot2) { final float v1 = values[slot1]; final float v2 = values[slot2]; - if (v1 > v2) { - return 1; - } else if (v1 < v2) { - return -1; - } else { - return 0; - } + return Float.compare(v1, v2); } @Override