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:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
