From 16f29edadf1b6082a44fffb31e7cf0c6585070a9 Mon Sep 17 00:00:00 2001 From: CheHyeonYeong Date: Sat, 28 Mar 2026 16:57:26 +0900 Subject: [PATCH 1/2] Fix incorrect assertion examples in DynamoDbIgnoreNulls Javadoc - Change empty() to anEmptyMap() for Map type checking - Fix nestedBean.getInnerBean2() to itemMap.get("innerBean2").m() - Fix typo: return innerBean -> return innerBean2 Signed-off-by: chehyeonyeong --- .../dynamodb/mapper/annotations/DynamoDbIgnoreNulls.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/services-custom/dynamodb-enhanced/src/main/java/software/amazon/awssdk/enhanced/dynamodb/mapper/annotations/DynamoDbIgnoreNulls.java b/services-custom/dynamodb-enhanced/src/main/java/software/amazon/awssdk/enhanced/dynamodb/mapper/annotations/DynamoDbIgnoreNulls.java index 51b71eba01cc..a7281308cc55 100644 --- a/services-custom/dynamodb-enhanced/src/main/java/software/amazon/awssdk/enhanced/dynamodb/mapper/annotations/DynamoDbIgnoreNulls.java +++ b/services-custom/dynamodb-enhanced/src/main/java/software/amazon/awssdk/enhanced/dynamodb/mapper/annotations/DynamoDbIgnoreNulls.java @@ -45,7 +45,7 @@ * } * * public AbstractBean getInnerBean2() { - * return innerBean; + * return innerBean2; * } * public void setInnerBean2(AbstractBean innerBean) { * this.innerBean2 = innerBean; @@ -63,10 +63,10 @@ * Map itemMap = beanTableSchema.itemToMap(bean, true); * * // innerBean1 w/ @DynamoDbIgnoreNulls does not have any attribute values because all the fields are null - * assertThat(itemMap.get("innerBean1").m(), empty()); + * assertThat(itemMap.get("innerBean1").m(), anEmptyMap()); * * // innerBean2 w/o @DynamoDbIgnoreNulls has a NULL attribute. - * assertThat(nestedBean.getInnerBean2(), hasEntry("attribute", nullAttributeValue())); + * assertThat(itemMap.get("innerBean2").m(), hasEntry("attribute", nullAttributeValue())); * } */ @SdkPublicApi From 7b7e5dda8f47276e0473820bbab2aa5f314454c1 Mon Sep 17 00:00:00 2001 From: Alex Woods Date: Fri, 3 Apr 2026 07:31:13 -0700 Subject: [PATCH 2/2] Minor wording update --- .../dynamodb/mapper/annotations/DynamoDbIgnoreNulls.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/services-custom/dynamodb-enhanced/src/main/java/software/amazon/awssdk/enhanced/dynamodb/mapper/annotations/DynamoDbIgnoreNulls.java b/services-custom/dynamodb-enhanced/src/main/java/software/amazon/awssdk/enhanced/dynamodb/mapper/annotations/DynamoDbIgnoreNulls.java index a7281308cc55..a7ff2baba727 100644 --- a/services-custom/dynamodb-enhanced/src/main/java/software/amazon/awssdk/enhanced/dynamodb/mapper/annotations/DynamoDbIgnoreNulls.java +++ b/services-custom/dynamodb-enhanced/src/main/java/software/amazon/awssdk/enhanced/dynamodb/mapper/annotations/DynamoDbIgnoreNulls.java @@ -62,10 +62,10 @@ * * Map itemMap = beanTableSchema.itemToMap(bean, true); * - * // innerBean1 w/ @DynamoDbIgnoreNulls does not have any attribute values because all the fields are null + * // innerBean1 with @DynamoDbIgnoreNulls does not have any attribute values because all the fields are null * assertThat(itemMap.get("innerBean1").m(), anEmptyMap()); * - * // innerBean2 w/o @DynamoDbIgnoreNulls has a NULL attribute. + * // innerBean2 without @DynamoDbIgnoreNulls has a NULL attribute. * assertThat(itemMap.get("innerBean2").m(), hasEntry("attribute", nullAttributeValue())); * } */