summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLester Solbakken <lesters@oath.com>2019-02-06 12:29:09 +0100
committerLester Solbakken <lesters@oath.com>2019-02-06 12:29:09 +0100
commit42bb163a8252ec9ee81b8c29eda7ca66b2a50594 (patch)
tree747e6c6a24bd0b0b7e3ba807276d793dea667a3e
parent245a9611bce4d9d214ccb76016b67b6ca441dd24 (diff)
Don't insert empty multivalued attribute fields in summary
-rw-r--r--searchsummary/src/vespa/searchsummary/docsummary/attributedfw.cpp6
1 files changed, 5 insertions, 1 deletions
diff --git a/searchsummary/src/vespa/searchsummary/docsummary/attributedfw.cpp b/searchsummary/src/vespa/searchsummary/docsummary/attributedfw.cpp
index 19dd096c46c..1ca1a336d2d 100644
--- a/searchsummary/src/vespa/searchsummary/docsummary/attributedfw.cpp
+++ b/searchsummary/src/vespa/searchsummary/docsummary/attributedfw.cpp
@@ -149,9 +149,13 @@ MultiAttrDFW::insertField(uint32_t docid, GetDocsumsState *state, ResType, Inser
using vespalib::slime::Cursor;
using vespalib::Memory;
const IAttributeVector & v = vec(*state);
- uint32_t entries = v.getValueCount(docid);
bool isWeightedSet = v.hasWeightedSetType();
+ uint32_t entries = v.getValueCount(docid);
+ if (entries == 0) {
+ return; // Don't insert empty fields
+ }
+
Cursor &arr = target.insertArray();
BasicType::Type t = v.getBasicType();
switch (t) {