msfroh commented on PR #13748: URL: https://github.com/apache/lucene/pull/13748#issuecomment-2339574918
I had another thought while walking home. Both of the anonymous CollectorManagers are the same (but produce slightly different collectors). What if you split it out into an inner class that takes a `Supplier<SimpleCollectorWithLeafReader>` as a constructor arg? That way the two checks can each have their anonymous implementations, but the anonymous nesting won't be as deep. -- 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
