reta commented on PR #13472:
URL: https://github.com/apache/lucene/pull/13472#issuecomment-2174215964

   > I don't think we can eliminate all the overhead of requiring some memory 
barriers for synchronising tasks. So maybe the problem eventually just is with 
tasks that are too small?
   
   This change is definitely an improvement (over the existing implementation)
   
   > I don't think we can eliminate all the overhead of requiring some memory 
barriers for synchronising tasks. So maybe the problem eventually just is with 
tasks that are too small?
   
   Eliminate - probably not, but reduce further - likely but it needs yet to be 
proven (as you rightly pointed out, `maybe the problem eventually just is with 
tasks that are too small?`, so optimizing further won't help). 


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