uschindler commented on PR #16382: URL: https://github.com/apache/lucene/pull/16382#issuecomment-4936276052
I think there is a use case. Many of our query parsers by default support wildcards, also the default Solr one (e.g., dismax). If it uses way too much memory for degenerate cases, it should be handled in the most ideal case. The code introduced here is just the default pattern used for removing duplicates in such parsers. Yes, it is an additional state variable. Maybe it can be similified a bit so you don't have the boolean setter everywhere. If you have an idea, apply it to the patch. Otherwise for me it is fine! -- 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]
