DynamoDB Enhanced Client: @DynamoDbIgnore fails in some property mapping scenarios

[hiroki-otaka]

Describe the bug

The DynamoDB Enhanced Client fails to recognize a valid setter when a bean has both getA() and isA() methods, even when isA() is annotated with @DynamoDbIgnore.

Regression Issue

• Select this option if this issue appears to be a regression.

Expected Behavior

a should be mapped using:

getA(): int
setA(int): void

and isA() should be ignored because of @DynamoDbIgnore.

Current Behavior

The setA(int) setter is not recognized, and the a attribute is not mapped correctly.

This appears to be caused by the interaction between JavaBeans property introspection and the two getters:

getA(): int
isA(): boolean

Even though isA() is annotated with @DynamoDbIgnore, it still affects property discovery.

Reproduction Steps

@DynamoDbBean
public class Foo {
    private int a;

    @DynamoDbAttribute("A")
    public int getA() {
        return a;
    }

    public void setA(int a) {
        this.a = a;
    }

    @DynamoDbIgnore
    public boolean isA() {
        return false;
    }
}

Possible Solution

No response

Additional Information/Context

The same class is mapped correctly by the AWS SDK for Java v1, which does not rely on the same JavaBeans property conventions as the v2 Enhanced Client.

AWS Java SDK version used

2.51.3

JDK version used

11.0.32.1

Operating System and version

Windows 11

[debora-ito]

@hiroki-otaka thank you for reaching out. I can reproduce.

Yes, it looks like the root cause is how the Java Beans Introspector handles conflict when two read methods of a property exist, it gives isA() priority over getA() but because it's for a boolean type it doesn't recognize setA(int) as the property writer.

And I understand how this feels like a regression because the same class worked in v1 DynamoDB Mapper, but we don't classify differences between v1 and v2 as regressions. V2 is a new implementation, differences in behavior can happen and should be documented in the Dev Guide and Javadoc.

Tracking this as a bug. Not ideal, but the workaround is to rename isA() to something that doesn't conflict with the property name.