drempapis opened a new pull request, #15961: URL: https://github.com/apache/lucene/pull/15961
This change continues the determinization cleanup started for regexp queries ([#15939](https://github.com/apache/lucene/pull/15939)) by applying the same model to wildcard queries. Previously, wildcard automata were implicitly determinized up front. After removing that implicit determinization, some code paths (especially query visiting/highlighting and equality checks) still assumed a DFA-only representation. `QueryVisitor.consumeTermsMatching()` accepted only `ByteRunAutomaton` (DFA). After removing implicit wildcard determinization, many valid query automata are NFA-backed. Forcing them back into `ByteRunAutomaton` at visit time would reintroduce determinization cost and defeat the goal of this work. This PR adds `QueryVisitor.consumeTermsMatchingRunnable()`, allowing visitor consumers (e.g. highlighter) to work with either DFA or NFA-backed execution via `ByteRunnable`, while keeping the existing `consumeTermsMatching()` API for compatibility. -- 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]
