jackylee-ch opened a new pull request, #9280:
URL: https://github.com/apache/paimon/pull/9280

   ### Purpose
   
   `RangeBitmapFileIndex#visitTopN` sets `strict = orders.size() == 1`: with 
more orders the
   index only prunes, so rows tied on the first key must survive for the engine 
to break the
   tie. #6178 threaded `strict` into `BitSliceIndexBitmap` but not into 
`fillNulls`, so a tied
   null group is still trimmed. Value ties already keep the group.
   
   `ORDER BY col1 ASC NULLS FIRST, col2 DESC LIMIT 2`, `col1 = 
[null,null,null,7,8]`:
   
   ```
   was  {0,1}    -> engine can only answer rows 1,0
   now  {0,1,2}  -> rows 2,1, correct
   ```
   
   ### Tests
   
   `RangeBitmapFileIndexTest#testTopNWithMultipleColumnsKeepsNullTies`, also 
pinning that a
   single order still trims.
   
   Written with Claude Code; reasoning and verification are mine.
   


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

Reply via email to