[ https://issues.apache.org/jira/browse/LUCENE-4120?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13292885#comment-13292885 ]
Dawid Weiss commented on LUCENE-4120: ------------------------------------- I looked at the patch and it looks good to me but I didn't really analyze it in-depth. As for fst packing, the idea is fairly simple -- you reduce the overall size of the fst by moving states which have lots incoming arcs to offsets which compress well (in vcoding). At least I think that's what Mike implemented (Mike is an unpredictable genius :) ). This presentation has some details: http://ciaa-fsmnlp-2011.univ-tours.fr/ciaa/upload/files/Weiss-Daciuk.pdf > FST should use packed integer arrays > ------------------------------------ > > Key: LUCENE-4120 > URL: https://issues.apache.org/jira/browse/LUCENE-4120 > Project: Lucene - Java > Issue Type: Improvement > Components: core/FSTs > Reporter: Adrien Grand > Assignee: Adrien Grand > Priority: Minor > Fix For: 4.0 > > Attachments: LUCENE-4120.patch > > > There are some places where an int[] could be advantageously replaced with a > packed integer array. > I am thinking (at least) of: > * FST.nodeAddress (GrowableWriter) > * FST.inCounts (GrowableWriter) > * FST.nodeRefToAddress (read-only Reader) > The serialization/deserialization methods should be modified too in order to > take advantage of PackedInts.get{Reader,Writer}. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org