diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2017-09-01 09:13:44 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-09-01 09:13:44 +0200 |
commit | c48afc03ef41b4f7b21f7a261b3a910f8c10e280 (patch) | |
tree | af3c7edd986b80f1bf4f88d5fdc08a73cd64aaf8 /container-search | |
parent | 37643456fd6cda1960decaa64f3a8df9b99264a0 (diff) |
Revert "render infinite or NaN relevance as "0.0""
Diffstat (limited to 'container-search')
3 files changed, 3 insertions, 7 deletions
diff --git a/container-search/src/main/java/com/yahoo/search/rendering/JsonRenderer.java b/container-search/src/main/java/com/yahoo/search/rendering/JsonRenderer.java index dd8e261a6b7..c39f0387d46 100644 --- a/container-search/src/main/java/com/yahoo/search/rendering/JsonRenderer.java +++ b/container-search/src/main/java/com/yahoo/search/rendering/JsonRenderer.java @@ -488,8 +488,7 @@ public class JsonRenderer extends AsynchronousSectionedRenderer<Result> { if (id != null) generator.writeStringField(ID, id); - generator.writeFieldName(RELEVANCE); - generator.writeNumber(hit.getRelevance().toString()); + generator.writeNumberField(RELEVANCE, hit.getRelevance().getScore()); if (hit.types().size() > 0) { // TODO: Remove types rendering on Vespa 7 generator.writeArrayFieldStart(TYPES); diff --git a/container-search/src/main/java/com/yahoo/search/result/Relevance.java b/container-search/src/main/java/com/yahoo/search/result/Relevance.java index 2f6967ee794..7737b01cc14 100644 --- a/container-search/src/main/java/com/yahoo/search/result/Relevance.java +++ b/container-search/src/main/java/com/yahoo/search/result/Relevance.java @@ -47,9 +47,6 @@ public class Relevance implements Comparable<Relevance> { */ @Override public String toString() { - if (Double.isNaN(score) || Double.isInfinite(score)) { - return "0.0"; - } return DoubleFormatter.stringValue(score); } diff --git a/container-search/src/test/java/com/yahoo/prelude/searcher/test/QuerySnapshotSearcherTestCase.java b/container-search/src/test/java/com/yahoo/prelude/searcher/test/QuerySnapshotSearcherTestCase.java index b17fdbb4cf2..b1763471c82 100644 --- a/container-search/src/test/java/com/yahoo/prelude/searcher/test/QuerySnapshotSearcherTestCase.java +++ b/container-search/src/test/java/com/yahoo/prelude/searcher/test/QuerySnapshotSearcherTestCase.java @@ -26,8 +26,8 @@ public class QuerySnapshotSearcherTestCase extends junit.framework.TestCase { Searcher searcher=new QuerySnapshotSearcher(); Result result = doSearch(searcher, new Query(), 0,10); Hit hit=result.hits().get(0); - assertEquals(Double.POSITIVE_INFINITY, hit.getRelevance().getScore()); - assertEquals("0.0", hit.getRelevance().toString()); + assertEquals(String.valueOf(Double.POSITIVE_INFINITY), + hit.getRelevance().toString()); } private Result doSearch(Searcher searcher, Query query, int offset, int hits) { |