uschindler commented on PR #14901: URL: https://github.com/apache/lucene/pull/14901#issuecomment-3045546980
Ok, so the vectorSize is missing in the test reproducer line. Anyways, thats an old issue. My problem with Robert's patch is that he removed the second system property `tests.forceintegervectors`. This allowed us to test integer vectors correctness. To keep the sysprop away, my proposal would be: **Whenever somebody sets the "bitsize" we should NOT bail out and execute the tests in TestVectorUtil.** -- 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: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
