diff options
Diffstat (limited to 'container-search/src/main/java/com/yahoo/prelude/fastsearch/FastHit.java')
-rw-r--r-- | container-search/src/main/java/com/yahoo/prelude/fastsearch/FastHit.java | 23 |
1 files changed, 9 insertions, 14 deletions
diff --git a/container-search/src/main/java/com/yahoo/prelude/fastsearch/FastHit.java b/container-search/src/main/java/com/yahoo/prelude/fastsearch/FastHit.java index 3bf10295ed9..b5e5ed9ed8f 100644 --- a/container-search/src/main/java/com/yahoo/prelude/fastsearch/FastHit.java +++ b/container-search/src/main/java/com/yahoo/prelude/fastsearch/FastHit.java @@ -22,6 +22,8 @@ import java.util.Map; import java.util.NoSuchElementException; import java.util.Set; import java.util.function.BiConsumer; +import java.util.logging.Level; +import java.util.logging.Logger; /** * A regular hit from a Vespa backend @@ -31,6 +33,8 @@ import java.util.function.BiConsumer; */ public class FastHit extends Hit { + private static final Logger log = Logger.getLogger(FastHit.class.getName()); + private static final byte[] emptyGID = new byte[GlobalId.LENGTH]; /** The index of the content node this hit originated at */ private int distributionKey; @@ -67,35 +71,26 @@ public class FastHit extends Hit { * Creates an empty and temporarily invalid summary hit */ public FastHit() { - super(new Relevance(0.0)); - globalId = emptyGID; - partId = 0; - distributionKey = 0; + this(emptyGID, 0.0, 0, 0); } public FastHit(byte[] gid, double relevance, int partId, int distributionKey) { this(gid, new Relevance(relevance), partId, distributionKey); } + public FastHit(byte[] gid, Relevance relevance, int partId, int distributionKey) { super(relevance); this.globalId = gid; this.partId = partId; this.distributionKey = distributionKey; + if (distributionKey < 0) + log.log(Level.WARNING, "Distribution key is negative: " + this, new RuntimeException()); } // Note: This constructor is only used for tests, production use is always of the empty constructor public FastHit(String uri, double relevancy) { - this(uri, relevancy, null); - } - - // Note: This constructor is only used for tests, production use is always of the empty constructor - private FastHit(String uri, double relevance, String source) { - super(new Relevance(relevance)); - partId = 0; - distributionKey = 0; - globalId = emptyGID; + this(emptyGID, relevancy, 0, 0); setId(uri); - setSource(source); types().add("summary"); } |