samredai opened a new pull request #4021:
URL: https://github.com/apache/iceberg/pull/4021


   This incorporates some of the post-merge feedback for PR #3691, namely, it 
adds an `InputStream` and `OutputStream` protocol which are used as the return 
types for `InputFile.open()` and `OutputFile.create()`.
   
   Using a `typing.Protocol` (thanks for the suggestion @emkornfield) allows 
for flexibility of these implementations. Validation is essentially structural 
subtyping (aka "static duck typing"). This will be very handy when developing 
FileIO implementations that heavily rely on third party libraries. It should 
allow us to maintain the interface without having to generate wrapper classes 
for the file-like objects provided by a third party library.


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

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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to