jackye1995 commented on a change in pull request #1573: URL: https://github.com/apache/iceberg/pull/1573#discussion_r518973894
########## File path: LICENSE ########## @@ -248,6 +248,8 @@ License: http://www.apache.org/licenses/LICENSE-2.0 This product includes code from Presto. * Retry wait and jitter logic in Tasks.java +* S3FileIO logic derived from PrestoS3FileSystem.java in S3InputStream.java Review comment: At this point of time, I don't see much similarity between the input and output stream of `PrestoS3FileSystem` and the ones here. Should we still have this line in license? ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
