uschindler commented on issue #11838: URL: https://github.com/apache/lucene/issues/11838#issuecomment-1271537616
For AutomatonQuery it is also required to rewrite, because there are different implementations used. When you expand to a BooleanQuery or TermInSetQuery, you need to do that in the rewrite phase. My question is: What brings moving it to the Weight? It is still not multithreaded! Maybe a good idea would be to allow the heavy calculations in Weight and also Rewrite to execute in an Executor. By default we pass a SameThreadExecutor implementation (basically the executor just calls `run()`), but a query implementation might use multiple threads by splitting the work into smaller pieces. E.g. a BooleanQuery could rewrite all its clauses in a separate task. -- 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
