Don't print document summary field for an empty raw field.

2 files changed, 5 insertions, 2 deletions

diff --git a/searchsummary/src/vespa/searchsummary/docsummary/check_undefined_value_visitor.cpp b/searchsummary/src/vespa/searchsummary/docsummary/check_undefined_value_visitor.cpp
index 24527e246cd..624efd5d834 100644
--- a/searchsummary/src/vespa/searchsummary/docsummary/check_undefined_value_visitor.cpp
+++ b/searchsummary/src/vespa/searchsummary/docsummary/check_undefined_value_visitor.cpp
@@ -77,8 +77,11 @@ CheckUndefinedValueVisitor::visit(const document::PredicateFieldValue&)
 }
 
 void
-CheckUndefinedValueVisitor::visit(const document::RawFieldValue&)
+CheckUndefinedValueVisitor::visit(const document::RawFieldValue& value)
 {
+    if (value.getValueRef().empty()) {
+        _is_undefined = true;
+    }
 }
 
 void
diff --git a/searchsummary/src/vespa/searchsummary/docsummary/check_undefined_value_visitor.h b/searchsummary/src/vespa/searchsummary/docsummary/check_undefined_value_visitor.h
index f2a769d4002..e652d5d122b 100644
--- a/searchsummary/src/vespa/searchsummary/docsummary/check_undefined_value_visitor.h
+++ b/searchsummary/src/vespa/searchsummary/docsummary/check_undefined_value_visitor.h
@@ -25,7 +25,7 @@ class CheckUndefinedValueVisitor : public document::ConstFieldValueVisitor
     void visit(const document::LongFieldValue&) override;
     void visit(const document::MapFieldValue& value) override;
     void visit(const document::PredicateFieldValue&) override;
-    void visit(const document::RawFieldValue&) override;
+    void visit(const document::RawFieldValue& value) override;
     void visit(const document::ShortFieldValue&) override;
     void visit(const document::StringFieldValue& value) override;
     void visit(const document::StructFieldValue&) override;