rahulgoswami opened a new pull request, #15454:
URL: https://github.com/apache/lucene/pull/15454

   
   
   ### Description
   
   #14607 advanced the SegmentInfos version to VERSION_86 (10) in header check 
during readCommit() to allow tests to find backward codecs until 9.x while 
throwing a graceful IndexFormatTooOldException for indexes < v8.6.0 (since by 
default anything < 10x is "too old"). 
   Earlier with VERSION_74 (9), readCodec() would throw an 
IllegalArgumentException since readCommit now checks for segment level 
compatibility instead of looking at when the index was first created, and would 
find the 7.x codecs missing for some of the old indexes. This was causing some 
of the tests to fail. 
   
   However  #15431 made me realize that we can afford to allow binary 
readability on 8.x indexes like was the case previously and also keep the tests 
happy., This change would also help keep the core logic and constraints 
consistent across 10x and main.
   


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