PR FIXUP - Tweak currentKV

Is technically more correct, and unlikely to have a meaningful impact on performance
sourcenetwork · Dec 12, 2024 · 0d096c1 · 0d096c1
1 parent 79a5013
commit 0d096c1
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/internal/db/fetcher/document.go b/internal/db/fetcher/document.go
@@ -106,11 +106,13 @@ func (f *documentFetcher) NextDoc() (immutable.Option[string], error) {
 			return immutable.None[string](), err
 		}
 
-		if dsKey.DocID != f.currentKV.Key.DocID {
-			f.currentKV = keyValue{
-				Key:   dsKey,
-				Value: res.Value,
-			}
+		previousKV := f.currentKV
+		f.currentKV = keyValue{
+			Key:   dsKey,
+			Value: res.Value,
+		}
+
+		if dsKey.DocID != previousKV.Key.DocID {
 			break
 		}
 	}