Allow attaching MetricPublisher for DynamoDBEnhanced and DynamoDBEnhancedAsyncClient client calls

[swaranga]

Today, when I use the low level DynamoDBClient or DynamoDBAsyncClient, I can attach a MetricsPublisher to the underlying GetItemRequest, PutItemRequest etc. However if I decide to use the DynamoDBEnhancedClient or DynamoDBEnhancedAsyncClient, there is no way to attach similar MetricsPublisher instances to load and save calls.

I am aware that I can attach a MetricsPublisher to the underlying DynamoDBClient instance used by the DynamoDBEnhanceClient but I need to collect and publish metrics at each request level.

This is similar to how for normal GetItem calls I can attach MetricPublishers at individual request level.

[debora-ito]

I see, DynamoDB Enhanced requests don't have the overrideConfiguration option.
Not sure exactly which load and save calls you mean though.

Marking as a feature request.

[swaranga]

Sorry, the load and save calls were actually in the V1 DynamoDBMapper API. In the V2 enhanced client, they are more like getItem, putItem: https://aws.amazon.com/blogs/developer/introducing-enhanced-dynamodb-client-in-the-aws-sdk-for-java-v2/. And yes, the ability to do an overrideConfiguration for each request will solve this problem.

[swaranga]

Ping. Any updates?

[swaranga]

Ping. Any updates on this? Actually, do you take pull requests?

[debora-ito]

Yes, we do take pull requests.