Re: [PR] AWS: Add url-connection-client to aws-bundle [iceberg]

2024-03-04 Thread via GitHub
nastra closed pull request #9685: AWS: Add url-connection-client to aws-bundle URL: https://github.com/apache/iceberg/pull/9685 -- 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.

Re: [PR] AWS: Add url-connection-client to aws-bundle [iceberg]

2024-03-03 Thread via GitHub
danielcweeks commented on PR #9685: URL: https://github.com/apache/iceberg/pull/9685#issuecomment-1975284396 I don't think we can include this by default. The issue is if client is ever created without explicitly setting the HTTP implementation and there are two implementations on the class

Re: [PR] AWS: Add url-connection-client to aws-bundle [iceberg]

2024-03-03 Thread via GitHub
ismailsimsek commented on PR #9685: URL: https://github.com/apache/iceberg/pull/9685#issuecomment-1975274866 +1 adding it. without this dependency getting following error `Caused by: java.lang.NoClassDefFoundError: com/amazonaws/event/ProgressListener` -- This i

Re: [PR] AWS: Add url-connection-client to aws-bundle [iceberg]

2024-02-15 Thread via GitHub
94Sip commented on PR #9685: URL: https://github.com/apache/iceberg/pull/9685#issuecomment-1946981814 I have used v1.4.3 with Glue jobs and simply referenced the url connection client in the --extra-jars job parameter and had no problems, and I used the `http-client.type=urlconnection` Spar

Re: [PR] AWS: Add url-connection-client to aws-bundle [iceberg]

2024-02-08 Thread via GitHub
nastra commented on PR #9685: URL: https://github.com/apache/iceberg/pull/9685#issuecomment-1934536087 I'll hold off merging this because there might be a separate issue when bundling this dependency according to @danielcweeks -- This is an automated message from the Apache Git Service.