dungba88 commented on code in PR #12624:
URL: https://github.com/apache/lucene/pull/12624#discussion_r1415666711
##########
lucene/core/src/java/org/apache/lucene/util/fst/FSTCompiler.java:
##########
@@ -218,13 +279,19 @@ public Builder<T> allowFixedLengthArcs(boolean
allowFixedLengthArcs) {
}
/**
- * How many bits wide to make each byte[] block in the BytesStore; if you
know the FST will be
- * large then make this larger. For example 15 bits = 32768 byte pages.
+ * Set the {@link DataOutput} which is used for low-level writing of FST.
If you want the FST to
+ * be immediately readable, you need to use a DataOutput that also
implements {@link FSTReader},
Review Comment:
I think when we change the compile() to only return metadata, users need to
create the FST with the on-heap FSTReader, and thus it needs to be public.
--
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]
