diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2018-05-22 18:22:39 +0200 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2018-05-23 21:09:53 +0200 |
commit | 6c8951fe8a35d2644f2160e1245282566c045cce (patch) | |
tree | e39bafd23af91e050c483f70f73fb7de5f3a9a72 /container-search | |
parent | b4c468dcc75260a09a6cb6b2b6f1a279878228ca (diff) |
Avoid expensive fields() method call that creates temporary map.
Diffstat (limited to 'container-search')
-rw-r--r-- | container-search/src/main/java/com/yahoo/prelude/fastsearch/FastHit.java | 7 |
1 files changed, 3 insertions, 4 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 bffc0317df7..c517742f0e5 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 @@ -98,10 +98,9 @@ public class FastHit extends Hit { if (uri != null) return uri; // TODO: Remove on Vespa 7, this should be one of the last vestiges of URL field magic - if (fields().containsKey("uri")) { - // trigger decoding - Object o = getField("uri"); - setId(o.toString()); + Object uriField = getField("uri"); + if (uriField != null) { + setId(uriField.toString()); return super.getId(); } |