uschindler commented on PR #14901: URL: https://github.com/apache/lucene/pull/14901#issuecomment-3045521943
> > I think the test should fail due to the IllegalArgumentException anyways. There is some stupid bullshit going on. Verbose or not should both create the same result. > > It's not about tests.verbose - it's about the seed. This code is either hit or not, depending on the main tests.seed - some other randomized, derived, parameters are affecting it. Exactly! When running it multiple times it fails 1/3 of it.... If the randomly assigned vector bitsize does not match expectations it fails. The problem with this PR is: We don't test the integer vectors anymore because Robert removed the "forceIntegerVectors" flag. I think it should stay alive (so please revert removal of the sysprop). -- 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
