Shibi Balamurugan

> new generic list factory methods ConjureCollections which do not allow null values + update codegen to use them @carterkozak Just to clarify this, should these new non-null ConjureCollections methods...

@jainankitk ah linked the wrong PR. It was https://github.com/apache/lucene/pull/13472 that made this change, and it seems like there were considerable speedups. Though it comes at the cost of making fair...

This really only impacts Knn queries where the work is done in rewrite and indexSearcher.getTaskExecutor() is called from the knn query. For all other queries, the caller can control what...

@jpountz `IMO Lucene should own how queries execute concurrently instead of making it pluggable` then why allow an executor service to be passed in? Previously, lucene didn't have control of...

@javanna I think what users need (or expect) with a TaskExecutor is to be able to have the created tasks run on the ExecutorService provided by the user. In my...

@javanna Hey! I kinda disagree with: `general idea was to make concurrency as transparent as possible` since the caller thread is used alongside another exec service. I think trading off...

@javanna not sure about the underlying implementation of these queries. Does using intra-segment partitioning negatively impact these queries (TermInSet/MultiTerm) or is there still some benefit from using the feature?