dweiss opened a new issue, #13820: URL: https://github.com/apache/lucene/issues/13820
### Description The change in #12841 brought this pair of methods to DataInput: ``` public final void readGroupVInts(long[] dst, int limit) throws IOException { ... } protected void readGroupVInt(long[] dst, int offset) throws IOException { ... } ``` If one wishes to write a DataInput delegating to another, existing DataInput, it's no longer possible because the public method is final and the protected method cannot be invoked on the delegate (cross-package access). A corresponding method in DataOutput is not final. I would remove the final modifier from the API for symmetry. Thoughts? ### Version and environment details _No response_ -- 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. To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org