jainankitk commented on PR #15383:
URL: https://github.com/apache/lucene/pull/15383#issuecomment-3530421026

   > Reading from a single `FixedBitSet` can be done by multiple threads with 
no synchronization. It's just reading from a long[].
   
   I guess we can do that, but still not sure if it will seamlessly integrate 
into existing abstractions on top of that. I was initially thinking about say 
cost function of this iterator, but there seems to be implementation for 
specific docId range `cardinality(int from, int to)`. But I am still concerned, 
there might be few other unknown things that might pop up.
   
   Also from the performance perspective, even simple iteration on this 
`long[]` will be randomized due to different threads accessing different parts 
of the array. So, it might be efficient to partition into `long[][]` where each 
row is accessed sequentially by one thread


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to