summaryrefslogtreecommitdiffstats
path: root/searchcore
diff options
context:
space:
mode:
authorArne H Juul <arnej@yahooinc.com>2021-10-10 10:36:03 +0000
committerArne H Juul <arnej@yahooinc.com>2021-10-10 11:54:58 +0000
commita16c551ff5ba407cdd8c4d7136f0a15ef0f3d078 (patch)
tree28a5d70fd95e1feb6c394e228ee0b1e36ca75cdd /searchcore
parent2c5c43c14dbc62d83e6e4fe3de6695423f91cdd9 (diff)
Revert "wait with actual reporting until we want errors"
This reverts commit a45f09c5edf40fdf01d18bfb92807c59f73defa3.
Diffstat (limited to 'searchcore')
-rw-r--r--searchcore/src/vespa/searchcore/grouping/groupingmanager.cpp7
1 files changed, 1 insertions, 6 deletions
diff --git a/searchcore/src/vespa/searchcore/grouping/groupingmanager.cpp b/searchcore/src/vespa/searchcore/grouping/groupingmanager.cpp
index 863511da5ee..42c5d16162e 100644
--- a/searchcore/src/vespa/searchcore/grouping/groupingmanager.cpp
+++ b/searchcore/src/vespa/searchcore/grouping/groupingmanager.cpp
@@ -61,12 +61,7 @@ GroupingManager::init(const IAttributeContext &attrCtx)
list.push_back(groupingList[i]);
} catch (const std::exception & e) {
auto msg = fmt("Could not locate attribute for grouping number %ld : %s. Ignoring grouping '%s'", i, e.what(), grouping.asString().c_str());
- //
- // NOTE: if this issue is reported as an error in the
- // search reply, the grouping searcher will discard all
- // grouping results, which may or may not be what we want.
- //
- // Issue::report(msg);
+ Issue::report(msg);
LOG(error, "%s", msg.c_str());
}
}