diff options
author | Jon Bratseth <bratseth@oath.com> | 2018-05-11 19:10:58 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-05-11 19:10:58 +0200 |
commit | 40e9964ea6f7e9940aebd45591822cb7982a914d (patch) | |
tree | ed9b0986dbe39e57c55f003679a07fbf9acc42a9 /container-search/src/main/java/com/yahoo/search/grouping | |
parent | cf80ff1b7ce08cd3fdfd8c789de45da02ffba5c9 (diff) |
Revert "Revert "Bratseth/allocation free hit field traversal""
Diffstat (limited to 'container-search/src/main/java/com/yahoo/search/grouping')
-rw-r--r-- | container-search/src/main/java/com/yahoo/search/grouping/result/HitRenderer.java | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/container-search/src/main/java/com/yahoo/search/grouping/result/HitRenderer.java b/container-search/src/main/java/com/yahoo/search/grouping/result/HitRenderer.java index 259b219b181..37906c8012f 100644 --- a/container-search/src/main/java/com/yahoo/search/grouping/result/HitRenderer.java +++ b/container-search/src/main/java/com/yahoo/search/grouping/result/HitRenderer.java @@ -13,7 +13,7 @@ import java.util.Map; /** * This is a helper class for rendering grouping results. * - * @author <a href="mailto:simon@yahoo-inc.com">Simon Thoresen</a> + * @author Simon Thoresen */ public abstract class HitRenderer { @@ -50,9 +50,7 @@ public abstract class HitRenderer { if (hit instanceof RootGroup) { renderContinuation(Continuation.THIS_PAGE, ((RootGroup)hit).continuation(), writer); } - for (String label : hit.fieldKeys()) { - writer.openTag(TAG_OUTPUT).attribute(ATR_LABEL, label).content(hit.getField(label), false).closeTag(); - } + hit.forEachField((name, value) -> writer.openTag(TAG_OUTPUT).attribute(ATR_LABEL, name).content(value, false).closeTag()); } else if (hit instanceof HitList) { writer.openTag(TAG_HIT_LIST).attribute(ATR_LABEL, ((HitList)hit).getLabel()); renderContinuations(((HitList)hit).continuations(), writer); |