iverase opened a new issue, #16052:
URL: https://github.com/apache/lucene/issues/16052

   Some APIs in LeafReader declared throws IOException even though, in Lucene’s 
own implementations they are actually not doing any I /O and probably should 
never do I /O. They provide quick access to some already load metadata for 
quickly decide if the index needs to be searched or not.
   
   It is difficult to tell from a developer point of view which methods are 
doing I /O and which are not because everything throws IOException. Therefore 
the proposal here is to remove the declared IOException from methods that 
should not be doing I /O.
   
   In particular I am thinking in this two methods, there might be more:
   
   ```java
   public abstract PointValues getPointValues(String field);
   
   public abstract Terms terms(String field);
   ```
   
   This is changing a public API so it should be a lucene 11 change only.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to