sohami commented on issue #12347: URL: https://github.com/apache/lucene/issues/12347#issuecomment-1591876384
@atris To summarize, there are 2 separate functionality I am looking to add: 1) Custom slice computation which the extension can provide. For this we can provide a constructor in `IndexSearcher` which takes in `LeafSlice` array from extension. I think probably there is no concern with this. 2) Mechanism to provide custom `SliceExecutor` implementation or deprecate this with some other mechanism. I would ideally like to provide a mechanism for extensions to be able to give a custom implementation of it. The default implementations takes into consideration certain limiting factor to apply back pressure which will not be needed in all the cases (as shared [above](https://github.com/apache/lucene/issues/12347#issuecomment-1589876811)) and will also simplify the reasoning behind which `slices` got executed on which thread-pool. So keeping the existing default and giving the flexibility to customize it is what I guess will be helpful here. This is still being discussed and would be great to hear your feedback as well. -- 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
