jpountz commented on code in PR #12908:
URL: https://github.com/apache/lucene/pull/12908#discussion_r1422523889
##########
lucene/core/src/java/org/apache/lucene/util/fst/FST.java:
##########
@@ -109,10 +109,20 @@ public enum INPUT_TYPE {
// Increment version to change it
private static final String FILE_FORMAT_NAME = "FST";
- private static final int VERSION_START = 6;
+
+ /** First supported version, this is the version that was used when
releasing Lucene 7.0. */
+ public static final int VERSION_START = 6;
+
private static final int VERSION_LITTLE_ENDIAN = 8;
Review Comment:
So version 7 was introduced by
https://github.com/apache/lucene/commit/5dd9c4c04bcab9911b9a9f0b092eadc50262ee2c
but it was later removed because it did not need version checks on the read
side for bw compat. I'll add a comment explaining the gap.
--
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
