DisableFetchingTableMetadata leads to key exception if PropertyConverter is used

[Dreamescaper]

Describe the bug

DisableFetchingTableMetadata does not account for PropertyConverters. See repro.

Expected Behavior

It shouldn't fail trying to save item with a strange message.

Current Behavior

System.InvalidOperationException: 'Key attribute Type must be of type Numeric'

Reproduction Steps

Repro

using Amazon.DynamoDBv2;
using Amazon.DynamoDBv2.DataModel;
using Amazon.DynamoDBv2.DocumentModel;
using Amazon.DynamoDBv2.Model;
using Amazon.Runtime;

var config = new AmazonDynamoDBConfig { ServiceURL = $"http://localhost:8000" };
var credentials = new BasicAWSCredentials("access", "secret");
var dynamoDbClient = new AmazonDynamoDBClient(credentials, config);
var context = new DynamoDBContext(dynamoDbClient, new DynamoDBContextConfig
{
    DisableFetchingTableMetadata = true
});

var tables = await dynamoDbClient.ListTablesAsync();
if (!tables.TableNames.Contains("TestTable"))
{
    await dynamoDbClient.CreateTableAsync(new CreateTableRequest
    {
        TableName = "TestTable",
        AttributeDefinitions =
        [
            new AttributeDefinition
            {
                AttributeName = "Type",
                AttributeType = ScalarAttributeType.S
            },
            new AttributeDefinition
            {
                AttributeName = "Id",
                AttributeType = ScalarAttributeType.S
            }
        ],
        KeySchema =
        [
            new KeySchemaElement
            {
                AttributeName = "Type",
                KeyType = KeyType.HASH
            },
            new KeySchemaElement
            {
                AttributeName = "Id",
                KeyType = KeyType.RANGE
            }
        ],
        ProvisionedThroughput = new ProvisionedThroughput
        {
            ReadCapacityUnits = 1,
            WriteCapacityUnits = 1
        }
    });

    Console.WriteLine("Table created");
}

await context.SaveAsync(new TestType
{
    Id = "1",
    Type = TypeEnum.Value2,
    Name = "Test",
});
Console.WriteLine("Item saved.");

var item = await context.LoadAsync<TestType>(TypeEnum.Value2, "1");
Console.WriteLine($"Item loaded: {item}");

[DynamoDBTable("TestTable")]
record class TestType
{
    [DynamoDBHashKey]
    [DynamoDBProperty(Converter = typeof(EnumAsStringConverter<TypeEnum>))]
    public TypeEnum Type { get; set; }

    [DynamoDBRangeKey]
    public string Id { get; set; }

    public string Name { get; set; }
}

enum TypeEnum
{
    Value1,
    Value2
}

public class EnumAsStringConverter<T> : IPropertyConverter where T : struct
{
    public object FromEntry(DynamoDBEntry entry) => Enum.Parse<T>(entry.AsString());
    public DynamoDBEntry ToEntry(object value) => new Primitive(value.ToString());
}

The code abode fails with an exception:
System.InvalidOperationException: 'Key attribute Type must be of type Numeric'

It works fine if I set DisableFetchingTableMetadata to false.

Possible Solution

No response

Additional Information/Context

No response

AWS .NET SDK and/or Package version used

Amazon.Lambda.DynamoDBEvents 3.7.302.19

Targeted .NET Platform

.NET 8

Operating System and version

AmazonLinuix

[ashishdhingra]

Needs review with the team.

[ashishdhingra]

The PR had been shipped AWSSDK.DynamoDBv2 version 3.7.302.26.

[github-actions]

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.