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

Reply via email to