mikemccand commented on pull request #2052:
URL: https://github.com/apache/lucene-solr/pull/2052#issuecomment-729002846


   > IMHO, we should add more Javadocs, that this directory may throw 
UnsupportedOperationException if the underlying platform does not allow direct 
IO.
   
   Hmm, maybe this is a good reason to make this change in two steps (add pure 
java version, then, later, remove native one)?  Which platforms will now throw 
`UnsupportedOperationException` with this new pure-java solution?


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

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