David Smiley created LUCENE-6011:
------------------------------------

             Summary: CompressingTermVectors should put checkDoc and 
checkPosition within assert
                 Key: LUCENE-6011
                 URL: https://issues.apache.org/jira/browse/LUCENE-6011
             Project: Lucene - Core
          Issue Type: Improvement
          Components: core/codecs
            Reporter: David Smiley
            Priority: Minor


CompressingTermVectorsReader.TVDocsEnum calls checkPosition() and checkDoc() 
but not from within asserts.  And nextPosition() has some leading checks, also 
not within asserts.  I believe these should all be within asserts.  This is 
low-level code that can get called a ton of times, particularly via 
TokenSources.getTokenStream.

I'd be happy to create a patch there is preliminary agreement.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to