gsmiller commented on PR #13669:
URL: https://github.com/apache/lucene/pull/13669#issuecomment-2297481110

   Looks good, thanks! I'm OK with merging this, but I want to also take a 
moment to ask that we're somewhat cautious with racking up lots of small 
changes like this that cannot be back-ported onto 9.x (which still uses java11 
as a min requirement, which doesn't support this style switch). Lots of 
divergences like this can make back-porting bug fixes or other impactful 
changes harder, so I'd like us to keep that in mind. Since we're close to a 
10.0 and last-9.x release, I'm OK with this though.


-- 
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

Reply via email to