msokolov opened a new pull request #753: LUCENE-8895: switch all FST usage to enable array-with-gaps encoding URL: https://github.com/apache/lucene-solr/pull/753 I'm not sure we want to commit this as-is, mainly because of the change to {{blocktreeords}} codec. With this change, if we encoded some node's outgoing Arcs as an array-with-gaps, we treat it as a list, scanning forward with a linear scan, rather than a binary search. Implementing binary search in this case might be possible, but it's quite complex. Another possibility is to leave the option of disabling the array-with-gaps opto and leaving it off for this one case. I'd like to get a sense how widely used this codec is and whether it's worth having that preserved in the API.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
