[ https://issues.apache.org/jira/browse/LUCENE-5156?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13730688#comment-13730688 ]
Adrien Grand commented on LUCENE-5156: -------------------------------------- Patch looks good, +1 > CompressingTermVectors termsEnum should probably not support seek-by-ord > ------------------------------------------------------------------------ > > Key: LUCENE-5156 > URL: https://issues.apache.org/jira/browse/LUCENE-5156 > Project: Lucene - Core > Issue Type: Bug > Reporter: Robert Muir > Attachments: LUCENE-5156.patch > > > Just like term vectors before it, it has a O(n) seek-by-term. > But this one also advertises a seek-by-ord, only this is also O(n). > This could cause e.g. checkindex to be very slow, because if termsenum > supports ord it does a bunch of seeking tests. (Another solution would be to > leave it, and add a boolean so checkindex never does seeking tests for term > vectors, only real fields). > However, I think its also kinda a trap, in my opinion if seek-by-ord is > supported anywhere, you kinda expect it to be faster than linear time...? -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators 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