javanna commented on PR #14524:
URL: https://github.com/apache/lucene/pull/14524#issuecomment-2865817224

   Thanks for clarifying @Shibi-bala . 
   
   I get a bit confused here with the mention of query fairness. What #13472 
does is advance execution on the caller thread, that would otherwise be waiting 
for the executor service's threads to complete their work, which is wasteful. 
At the same time it decreases the need for forking which provides better 
performance.
   
   I find it counter-intuitive that to provide more fairness we would go back 
to a model where the caller thread waits most of its time, and does a lot more 
forking than needed. Is there a way to implement the grouping of fairness you 
have in mind on top of the current TaskExecutor? What I am thinking is that 
despite it may execute on the caller thread, it does still execute tasks in a 
certain order.
   


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