danielcweeks commented on pull request #1573: URL: https://github.com/apache/iceberg/pull/1573#issuecomment-708580621
@jackye1995 and @kbendick I think you both make a good case for going forward with v2. After looking into the docs a little, the main point for me was that it provides more control over the underlying http transport, which we may want to take advantage of. The usage in the initial version is pretty basic and should be trivial to switch over to v2 (I have no objection to doing that). The main issue is testing support for the s3mock library with junit4 doesn't appear to support v2 so we would need to switch to junit5. (Is there better testing harnesses in/for v2?). @rdblue do you have any objection to using junit5 for this module? ---------------------------------------------------------------- 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]
