Pulkitg64 commented on code in PR #16415:
URL: https://github.com/apache/lucene/pull/16415#discussion_r3677484114


##########
lucene/test-framework/src/java/org/apache/lucene/tests/index/BaseKnnVectorsFormatTestCase.java:
##########
@@ -2071,6 +2071,94 @@ public void testReadQuantizedVectorWithEmptyRawVectors() 
throws Exception {
     }
   }
 
+  /**
+   * Tests that dropping the raw full-precision vectors does not change 
scoring. {@link
+   * FloatVectorValues#scorer(float[])} is documented to score against the 
quantized vectors when
+   * the underlying format quantizes, so a quantized index must produce 
identical scores before and
+   * after its raw vector file is emptied.
+   */
+  public void testScoresUnchangedWithEmptyRawVectors() throws Exception {

Review Comment:
   Yes, the test is failing without the fix. Sample Error:
   
   ```
   TestLucene104ScalarQuantizedVectorsFormat > 
testScoresUnchangedWithEmptyRawVectors FAILED
       java.lang.AssertionError: score changed for doc 0 after dropping raw 
vectors expected:<1.0> but was:<2.2135081>
   ```
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to