chenboat opened a new pull request, #15763:
URL: https://github.com/apache/pinot/pull/15763

   
   Add a unit test to verify the critical behavior of OrDocIdSet as stated in 
its javadoc comment:
   
   ```
   When there are more than one index-base BlockDocIdIterator 
(SortedDocIdIterator or BitmapBasedDocIdIterator),
    *     merge them and construct a BitmapDocIdIterator from the merged 
document ids. If there is no remaining
    *     BlockDocIdIterator, directly return the merged BitmapDocIdIterator; 
   ```
   
   Today the actual implementation in fact violates the above behavior as shown.


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