diff options
author | Jon Bratseth <bratseth@verizonmedia.com> | 2019-09-24 19:47:59 +0200 |
---|---|---|
committer | Jon Bratseth <bratseth@verizonmedia.com> | 2019-09-24 19:47:59 +0200 |
commit | ec62b8595306a40ac835aecf52fe9b03a0779a05 (patch) | |
tree | 0543168b850407922b82c78f8cfb734ed3e57382 /searchlib | |
parent | 2c424c78c538e6cc59ccf072960cad40f4bdc50e (diff) |
Handle NaN in hit aggregation sorting
Diffstat (limited to 'searchlib')
-rw-r--r-- | searchlib/src/main/java/com/yahoo/searchlib/aggregation/HitsAggregationResult.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/searchlib/src/main/java/com/yahoo/searchlib/aggregation/HitsAggregationResult.java b/searchlib/src/main/java/com/yahoo/searchlib/aggregation/HitsAggregationResult.java index c7ecdbd798d..b462fb67dd5 100644 --- a/searchlib/src/main/java/com/yahoo/searchlib/aggregation/HitsAggregationResult.java +++ b/searchlib/src/main/java/com/yahoo/searchlib/aggregation/HitsAggregationResult.java @@ -161,7 +161,7 @@ public class HitsAggregationResult extends AggregationResult { public void postMerge() { Collections.sort(hits, new Comparator<Hit>() { public int compare(Hit lhs, Hit rhs) { - return (lhs.getRank() > rhs.getRank()) ? -1 : (lhs.getRank() < rhs.getRank()) ? 1 : 0; + return -Double.compare(lhs.getRank(), rhs.getRank()); } }); if ((maxHits >= 0) && (hits.size() > maxHits)) { |