uschindler commented on PR #12417: URL: https://github.com/apache/lucene/pull/12417#issuecomment-1627011417
> Thank you, this worked. I think we can try using lucene-util to test the performance of this draft PR, as @jpountz suggested in #12396 . If we want to compare the vector and scalar versions, which script should we run in lucene-util? Hi, The problem is you want three variants: - old index format (scalar) - new index format scalar - new index format vectorized For the old vs new scalar the indexes are different this needs special handling. Luceneutil shares by default the same index. Maybe you can also compare different indexes, but I don't know how. @mikemccand knows more. To compare vectorized and scalar new variant use same checkout directory in config and only pass the vector incubator command line in one config and default command line in the other. Be sure to download a large index (wikimedium at least) and configure it to run enough queries! If the performance tests are fine we need to cleanup here: make a copy of old codec on backwards codecs module. Rename current codec with your changes to Lucene98 and als rename the ForUtil classes (90 to 98). This is a bit work but -- 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
