OlivierJaquemet commented on PR #16446: URL: https://github.com/apache/lucene/pull/16446#issuecomment-5453535507
Thank you @epotyom for the time you took to look at this. I’m not sure I fully understand what you’re suggesting. I tried to keep the modifications from my PR while making the changes to `QueryParser.yy` as you suggested, but some tests are still failing, so I probably didn’t implement the change exactly as you intended. Would you be able to create another PR/branch that includes: * the new unit tests from my PR, * the modifications you have in mind, and * the new unit test scenario you’re proposing using the `EnglishAnalyzer`? That would help me better understand the expected changes and make sure I’m following the right approach. -- 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]
