diff options
author | Lester Solbakken <lesters@oath.com> | 2019-02-06 12:29:09 +0100 |
---|---|---|
committer | Lester Solbakken <lesters@oath.com> | 2019-02-06 12:29:09 +0100 |
commit | 42bb163a8252ec9ee81b8c29eda7ca66b2a50594 (patch) | |
tree | 747e6c6a24bd0b0b7e3ba807276d793dea667a3e /searchsummary/src | |
parent | 245a9611bce4d9d214ccb76016b67b6ca441dd24 (diff) |
Don't insert empty multivalued attribute fields in summary
Diffstat (limited to 'searchsummary/src')
-rw-r--r-- | searchsummary/src/vespa/searchsummary/docsummary/attributedfw.cpp | 6 |
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) { |