mosche commented on PR #26380: URL: https://github.com/apache/beam/pull/26380#issuecomment-1521212544
> However, when I reset AwsModule.java to the version from master branch, the tests from your version still pass. Is that intended? Just wonder if some tests are missing. The code removed in AwsModule is more or less irrelevant, that's just to log a warning to catch a bug in the older SDK version. If you reset the tests to master you will see some failures because initialization of the provider was changed to lazy init when first loading credentials. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
