jpountz commented on issue #12665:
URL: https://github.com/apache/lucene/issues/12665#issuecomment-1771171862

   >  It would somehow disable itself if the application sets its own static 
index sort?
   
   This is correct. This bit already works on the PR, IndexWriter doesn't check 
the new method that `OneMerge` got that allows reordering the resulting segment 
if an index sort is configured in the `IndexWriterConfig`.
   
   > It's odd/curious that PKLookup got slower
   
   The terms dictionary should be exactly the same, only doc IDs get 
renumbered. BlockTree optimizes for incremental IDs by storing the delta of a 
singleton doc ID with the previous doc ID (`termState.singletonDocID += 
BitUtil.zigZagDecode(l >>> 1);` in `Lucene90PostingsReader`). I wonder that it 
doesn't only help with storage, but also with lookups since it's stored as a 
single-byte vInt in the incremental case, but often on a variable number of 
bytes after reordering.


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