nastra commented on PR #12274: URL: https://github.com/apache/iceberg/pull/12274#issuecomment-2690310516
@anuragmantri as you already mentioned earlier, I think we should revert the AWS stuff in this PR. The problem is that `./gradlew :iceberg-aws:integrationTest` is never being executed as part of normal CI actions, meaning that we won't notice when those tests start to fail. Additionally, I believe the AWS integration tests actually need to have some proper ENV vars to fully pass (which we can't provide when those tests are running on GH actions) Moving the Azure tests to `integrationTest` as part of this PR is fine though -- 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
