Olivier L Applin
Olivier L Applin
Hello @MarkusKramer ! We think we might have a fix for this issue that would make it so `-initialize-at-run-time=software.amazon.awssdk.utils.cache.CachedSupplier` is not required, but I cannot seem to reproduce the problem...
@payne911 A possible solution could be to add an annotation to interface fields of the dtos, something like `@Implementations(classes = {ArrayList.class, LinkedList.class})` or `@Register({ArrayList.class, LinkedList.Class})` to specify right at the...
Beside the small nit, I posted everything looks fine. I launched a run of the Codebuild jobs for the unit-tests and waiting for them to succeed
The new added test `resolveCredentials_DefaultCredentialProviderWithReloadWhenModified` fails on jdk8 11 and 17 consistently after exhausting all 4 retries: ``` 16:51:31.970 [main] INFO software.amazon.awssdk.auth.credentials.DefaultCredentialsProviderTest - Credential file with content [[default], aws_access_key_id =...
> @L-Applin @anirudh9391 would it be possible to share CI build or complete logs of the build with latest changes? The fork PR #5259 doesn't have latest changes Sure, I...
Here is the relevant logs for the test failure: [log-events-viewer-result.txt](https://github.com/user-attachments/files/16445566/log-events-viewer-result.txt) This test also fails locally for me, tested on java 17. Isn't it failing for you?
> No, the tests are passing successfully in my local. I'm using Mac-Intel in my local.. Huh, that interesting, I'm on mac-m1. I'm not sure what could cause this discrepancy...
PR opened to address latest round of comments: https://github.com/aws/aws-sdk-java-v2/pull/5403
Please run the checkstyle plugin on `dynamodb-enhanced`, there is some checkstyle violations. When I do it by runing the following command `mvn org.apache.maven.plugins:maven-checkstyle-plugin:3.1.2:check -pl :dynamodb-enhanced` I get these violations: ```...
The test coverage checklist is not complete, could you comment on the test that aren't run to validate they are not required for this change?