bruno-roustant commented on a change in pull request #798: LUCENE-8906: Lucene50PostingsFormat.IntBlockTermState becomes public URL: https://github.com/apache/lucene-solr/pull/798#discussion_r307295236
########## File path: lucene/core/src/java/org/apache/lucene/codecs/lucene50/Lucene50PostingsFormat.java ########## @@ -453,15 +453,15 @@ public FieldsProducer fieldsProducer(SegmentReadState state) throws IOException } } - final static class IntBlockTermState extends BlockTermState { - long docStartFP = 0; - long posStartFP = 0; - long payStartFP = 0; - long skipOffset = -1; - long lastPosBlockOffset = -1; + public static final class IntBlockTermState extends BlockTermState { Review comment: You're right. I had hard time to fix all the javadoc for the precommit. I imposes to have a constructor doc since we have fields initialized with non 0 value. So I added a constructor. ---------------------------------------------------------------- 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