vismaytiwari commented on PR #16382: URL: https://github.com/apache/lucene/pull/16382#issuecomment-4934440312
Added the CHANGES entry under 11.0.0 → Optimizations (276e3c7). On the backport: it's doable. The collapse itself ports cleanly, and it's arguably more valuable on 10.x since `toAutomaton` there determinizes eagerly. The one wrinkle you flagged is real — 10.x's `toAutomaton(Term, int determinizeWorkLimit)` takes a work limit and determinizes inside (11.x dropped that), so the test would call `toAutomaton(term, DEFAULT_DETERMINIZE_WORK_LIMIT)` and skip the extra `determinize`. Small change. Happy to open the 10.x backport with that tweak + a 10.6.0 CHANGES entry if you'd like it there. Otherwise I think this is good to go from my side. -- 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]
