Merge pull request #33077 from vespa-engine/arnej/estimate-like-before

estimate should include docid 0 for consistency
vespa-engine · Jan 3, 2025 · f6acdd1 · f6acdd1
2 parents f5e1b12 + 4c7d852
commit f6acdd1
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/searchcore/src/vespa/searchcore/proton/documentmetastore/lid_allocator.cpp b/searchcore/src/vespa/searchcore/proton/documentmetastore/lid_allocator.cpp
@@ -228,7 +228,8 @@ class WhiteListBlueprint : public SimpleLeafBlueprint
           _lock(),
           _matchDataVector()
     {
-        setEstimate(HitEstimate(numActiveLids, false));
+        // account for DocId 0 (considered active for estimation purposes)
+        setEstimate(HitEstimate(numActiveLids + 1, false));
     }
 
     bool isWhiteList() const noexcept final { return true; }