Skip to content

Add benchmarking to DDB v2 mapper #6809

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
RanVaknin wants to merge 4 commits into
base: master
Choose a base branch
from

Fix marshalling

4331d39
Select commit
Loading
Failed to load commit list.
Draft

Add benchmarking to DDB v2 mapper #6809

Fix marshalling
4331d39
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Apr 6, 2026 in 15m 59s

Quality Gate failed

Failed conditions
2 Security Hotspots
18.3% Coverage on New Code (required ≥ 80%)
4.0% Duplication on New Code (required ≤ 3%)
D Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Annotations

Check warning on line 90 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/DynamoDBMapper.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Immediately return this expression instead of assigning it to the temporary variable "object". 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoJYNWh-T1cKmhw7&open=AZ0aKoJYNWh-T1cKmhw7&pullRequest=6809

Check warning on line 158 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/StandardModelFactories.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace the type specification in this constructor call with the diamond operator ("<>"). 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoJ3NWh-T1cKmhxa&open=AZ0aKoJ3NWh-T1cKmhxa&pullRequest=6809

Check warning on line 984 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/DynamoDBMapperConfig.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add the missing @deprecated Javadoc tag. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoKCNWh-T1cKmhyG&open=AZ0aKoKCNWh-T1cKmhyG&pullRequest=6809

Check warning on line 408 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/DynamoDBMapperTableModel.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Consider using the Consumer Builder method instead of creating this nested builder. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoI9NWh-T1cKmhv9&open=AZ0aKoI9NWh-T1cKmhv9&pullRequest=6809

Check warning on line 178 in http-clients/aws-crt-client/src/test/java/software/amazon/awssdk/http/crt/internal/CrtAsyncRequestExecutorTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this 'public' modifier. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1hRDWO8QJk5ZZv78gE&open=AZ1hRDWO8QJk5ZZv78gE&pullRequest=6809

Check warning on line 46 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/StandardBeanProperties.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace the type specification in this constructor call with the diamond operator ("<>"). 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoJmNWh-T1cKmhxB&open=AZ0aKoJmNWh-T1cKmhxB&pullRequest=6809

Check warning on line 103 in http-clients/aws-crt-client/src/test/java/software/amazon/awssdk/http/crt/internal/CrtAsyncRequestExecutorTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Join these multiple assertions subject to one assertion chain. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1hRDWO8QJk5ZZv78gI&open=AZ1hRDWO8QJk5ZZv78gI&pullRequest=6809

Check warning on line 39 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/StandardAnnotationMaps.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a private constructor to hide the implicit public one. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoLANWh-T1cKmhyl&open=AZ0aKoLANWh-T1cKmhyl&pullRequest=6809

Check warning on line 263 in services/s3/src/main/java/software/amazon/awssdk/services/s3/internal/multipart/UploadWithUnknownContentLengthHelper.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this empty statement. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1hRCfC8QJk5ZZv78fs&open=AZ1hRCfC8QJk5ZZv78fs&pullRequest=6809

Check warning on line 36 in http-clients/aws-crt-client/src/test/java/software/amazon/awssdk/http/crt/internal/InputStreamAdaptingHttpStreamResponseHandlerTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'org.junit.jupiter.api.BeforeEach'. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1hRDXe8QJk5ZZv78gc&open=AZ1hRDXe8QJk5ZZv78gc&pullRequest=6809

Check warning on line 589 in test/sdk-benchmarks/src/main/java/software/amazon/awssdk/benchmark/enhanced/dynamodb/V2MapperItemFactory.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Catch Exception instead of Throwable. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoDXNWh-T1cKmhve&open=AZ0aKoDXNWh-T1cKmhve&pullRequest=6809

Check warning on line 405 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/DynamoDBMapperTableModel.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Iterate over the "entrySet" instead of the "keySet". 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoI9NWh-T1cKmhv8&open=AZ0aKoI9NWh-T1cKmhv8&pullRequest=6809

Check warning on line 19 in http-clients/aws-crt-client/src/test/java/software/amazon/awssdk/http/crt/internal/CrtResponseHandlerTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'org.mockito.ArgumentMatchers.anyInt'. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1hRDW38QJk5ZZv78gK&open=AZ1hRDW38QJk5ZZv78gK&pullRequest=6809

Check warning on line 275 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/StandardTypeConverters.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unnecessary cast to "Converter". 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoJQNWh-T1cKmhwQ&open=AZ0aKoJQNWh-T1cKmhwQ&pullRequest=6809

Check warning on line 432 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/StandardAnnotationMaps.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract this nested try block into a separate method. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoLANWh-T1cKmhyj&open=AZ0aKoLANWh-T1cKmhyj&pullRequest=6809

Check warning on line 111 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/StandardBeanProperties.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Declare "setter" on a separate line. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoJmNWh-T1cKmhxN&open=AZ0aKoJmNWh-T1cKmhxN&pullRequest=6809

Check warning on line 188 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/ConvertibleType.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Merge this if statement with the enclosing one. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoKMNWh-T1cKmhyK&open=AZ0aKoKMNWh-T1cKmhyK&pullRequest=6809

Check warning on line 99 in test/sdk-benchmarks/src/main/java/software/amazon/awssdk/benchmark/apicall/httpclient/async/NettyHttpClientAlpnBenchmark.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this useless assignment to local variable "run". 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1hRDHH8QJk5ZZv78f5&open=AZ1hRDHH8QJk5ZZv78f5&pullRequest=6809

Check warning on line 386 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/DynamoDBMapperFieldModel.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoI1NWh-T1cKmhvn&open=AZ0aKoI1NWh-T1cKmhvn&pullRequest=6809

Check warning on line 375 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/StandardTypeConverters.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace the type specification in this constructor call with the diamond operator ("<>"). 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoJQNWh-T1cKmhwG&open=AZ0aKoJQNWh-T1cKmhwG&pullRequest=6809

Check warning on line 776 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/DynamoDBMapperConfig.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

A Singleton implementation was detected. Make sure the use of the Singleton pattern is required and the implementation is the right one for the context. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoKCNWh-T1cKmhxw&open=AZ0aKoKCNWh-T1cKmhxw&pullRequest=6809

Check warning on line 143 in http-clients/aws-crt-client/src/test/java/software/amazon/awssdk/http/crt/internal/CrtRequestExecutorTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this 'public' modifier. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1hRDYK8QJk5ZZv78gr&open=AZ1hRDYK8QJk5ZZv78gr&pullRequest=6809

Check warning on line 159 in services/s3/src/main/java/software/amazon/awssdk/services/s3/internal/multipart/UploadWithUnknownContentLengthHelper.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Change this instance-reference to a static reference. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1hRCfC8QJk5ZZv78fp&open=AZ1hRCfC8QJk5ZZv78fp&pullRequest=6809

Check failure on line 791 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/DynamoDBMapperConfig.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Save and re-use this "Random". 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoKCNWh-T1cKmhxy&open=AZ0aKoKCNWh-T1cKmhxy&pullRequest=6809

Check warning on line 954 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/DynamoDBMapperConfig.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not forget to remove this deprecated code someday. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoKCNWh-T1cKmhx5&open=AZ0aKoKCNWh-T1cKmhx5&pullRequest=6809
View more details on SonarQubeCloud