[GitHub] [lucene] uschindler commented on issue #11933: Remove IOContext from `Directory#openChecksumInput`?

[GitBox <gi...@apache.org>]

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

   Hi,
   I think the idea of IOContext was beyond simple flags like "sequentially reading" or "random IO" (and you can create your own IOContexts). I think we added that to allow full flexibility. For the checkum case it may be predefined, so yes, we could think of removing the argument. 
   
   I am +/-0 for removing it, on the other hand it does not hurt to have it for future extensibility. Do you have something in mind where it may hinder us?


-- 
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-unsubscribe@lucene.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org