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 2 commits into
base: master
Choose a base branch
from

Add tests

c1108e6
Select commit
Loading
Failed to load commit list.
Draft

Add benchmarking to DDB v2 mapper #6809

Add tests
c1108e6
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Apr 6, 2026 in 16m 6s

Quality Gate failed

Failed conditions
2 Security Hotspots
17.5% 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 213 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

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=AZ0aKoI9NWh-T1cKmhvt&open=AZ0aKoI9NWh-T1cKmhvt&pullRequest=6809

Check warning on line 41 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/DynamoDBTypeConverter.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=AZ0aKoJeNWh-T1cKmhw9&open=AZ0aKoJeNWh-T1cKmhw9&pullRequest=6809

Check warning on line 23 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 unused import 'java.util.AbstractMap.SimpleImmutableEntry'. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoJQNWh-T1cKmhw3&open=AZ0aKoJQNWh-T1cKmhw3&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 45 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/DynamoDBTypeConverter.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=AZ0aKoJeNWh-T1cKmhw-&open=AZ0aKoJeNWh-T1cKmhw-&pullRequest=6809

Check warning on line 42 in test/dynamodb-mapper-v2/src/main/java/software/amazon/awssdk/dynamodb/datamodeling/DynamoDBNativeBoolean.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=AZ0aKoIsNWh-T1cKmhvh&open=AZ0aKoIsNWh-T1cKmhvh&pullRequest=6809

Check warning on line 52 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-T1cKmhxD&open=AZ0aKoJmNWh-T1cKmhxD&pullRequest=6809

Check warning on line 180 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

Remove this use of "newInstance"; it is deprecated. 

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

Check warning on line 143 in http-clients/aws-crt-client/src/main/java/software/amazon/awssdk/http/crt/internal/response/CrtResponseAdapter.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=AZ1hRDSc8QJk5ZZv78f9&open=AZ1hRDSc8QJk5ZZv78f9&pullRequest=6809

Check warning on line 54 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

Remove this redundant "static" qualifier; nested enum types are implicitly static. 

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

Check warning on line 50 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

Remove this unused "properties" private field. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoI9NWh-T1cKmhwD&open=AZ0aKoI9NWh-T1cKmhwD&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 21 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.mockito.ArgumentMatchers.anyInt'. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ1hRDXe8QJk5ZZv78gU&open=AZ1hRDXe8QJk5ZZv78gU&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 386 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

Return an empty collection instead of null. 

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

Check warning on line 63 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

Remove this block of code, fill it in, or add a comment explaining why it is empty. 

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

Check warning on line 429 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

Move this method into "TypedMap". 

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

Check warning on line 618 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

Reorder the modifiers to comply with the Java Language Specification. 

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

Check failure on line 173 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

Refactor this method to reduce its Cognitive Complexity from 22 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=aws_aws-sdk-java-v2&issues=AZ0aKoKMNWh-T1cKmhyZ&open=AZ0aKoKMNWh-T1cKmhyZ&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-T1cKmhvp&open=AZ0aKoI1NWh-T1cKmhvp&pullRequest=6809

Check warning on line 236 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-T1cKmhv6&open=AZ0aKoI9NWh-T1cKmhv6&pullRequest=6809

Check warning on line 54 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

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=AZ0aKoJ3NWh-T1cKmhxd&open=AZ0aKoJ3NWh-T1cKmhxd&pullRequest=6809

Check warning on line 114 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=AZ1hRDYK8QJk5ZZv78gp&open=AZ1hRDYK8QJk5ZZv78gp&pullRequest=6809

Check warning on line 362 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

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=AZ0aKoI9NWh-T1cKmhvx&open=AZ0aKoI9NWh-T1cKmhvx&pullRequest=6809

Check warning on line 480 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

Declare "primitiveType" on a separate line. 

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