mikemccand commented on code in PR #12748:
URL: https://github.com/apache/lucene/pull/12748#discussion_r1383582139
##########
lucene/core/src/java/org/apache/lucene/util/fst/FST.java:
##########
@@ -96,6 +96,13 @@ public enum INPUT_TYPE {
*/
static final byte ARCS_FOR_DIRECT_ADDRESSING = 1 << 6;
+ /**
+ * Value of the arc flags to declare a node with continuous arcs designed
for pos the arc directly
+ * with labelToPos - firstLabel. like {@link #ARCS_FOR_BINARY_SEARCH} we use
flag combinations
+ * that will not occur at the same time.
+ */
+ static final byte ARCS_FOR_CONTINUOUS = ARCS_FOR_DIRECT_ADDRESSING +
ARCS_FOR_BINARY_SEARCH;
+
// Increment version to change it
private static final String FILE_FORMAT_NAME = "FST";
private static final int VERSION_START = 6;
Review Comment:
> I think so, we expect to throw `IndexFormatTooNewException` in`
CodecUtil.checkIndexHeader` when the old version code reading new index format.
so we should also bump the `VERSION_CURRENT` in `Lucene90BlockTreeTermsReader`?
+1 to also bump the version in `Lucene90BlockTreeTermsReader`.
--
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
