uschindler commented on issue #11912:
URL: https://github.com/apache/lucene/issues/11912#issuecomment-1309351422

   As discussed in chat an hour ago:
   - for MemorySegmentIndexInput it already has a reworked Exception code where 
also those suppress unused warnings are gone due to some trick: we just catch 
the exception, but pass them to a method that maps it to an user readable 
Lucene exception and regrow it  This reduced code duplication and removed most 
suppress warnings.
   
   The code in memory segments already contains a message to report negative 
positions. I can change it to show the actual position.
   
   I would backport the exception factory methods to the byte buffer index 
input, that's a bit mechanical work but makes the code also better readable.


-- 
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

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