Tony-X commented on issue #12696:
URL: https://github.com/apache/lucene/issues/12696#issuecomment-1775807115

   > In 11.0, remove all patching logic which will, a) simplify the code a bit, 
and b) remove the (likely minor) overhead on read of looking up the number of 
patches in a block, which is always 0.
   
   I like this think of it as an index-level property rather than a block-level 
property. We can just check once instead of doing it per-block. 
   
   Maybe write something in the index header to indicate if patching is there 
(default to yes - in 9.x ). Then new indexes will write additional header to 
indicate there is not patching in the whole index (10.x behavior). Then remove 
the header as well as the patching support altogether in 11.x.     


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