{"action":"created","issue":{"url":"https://api.github.com/repos/awslabs/dynamodb-cross-region-library/issues/46","repository_url":"https://api.github.com/repos/awslabs/dynamodb-cross-region-library","labels_url":"https://api.github.com/repos/awslabs/dynamodb-cross-region-library/issues/46/labels{/name}","comments_url":"https://api.github.com/repos/awslabs/dynamodb-cross-region-library/issues/46/comments","events_url":"https://api.github.com/repos/awslabs/dynamodb-cross-region-library/issues/46/events","html_url":"https://github.com/awslabs/dynamodb-cross-region-library/issues/46","id":239310344,"number":46,"title":"NoopCloudWatch does not implement putMetricDataAsync","user":{"login":"colinking","id":2907397,"avatar_url":"https://avatars3.githubusercontent.com/u/2907397?v=3","gravatar_id":"","url":"https://api.github.com/users/colinking","html_url":"https://github.com/colinking","followers_url":"https://api.github.com/users/colinking/followers","following_url":"https://api.github.com/users/colinking/following{/other_user}","gists_url":"https://api.github.com/users/colinking/gists{/gist_id}","starred_url":"https://api.github.com/users/colinking/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/colinking/subscriptions","organizations_url":"https://api.github.com/users/colinking/orgs","repos_url":"https://api.github.com/users/colinking/repos","events_url":"https://api.github.com/users/colinking/events{/privacy}","received_events_url":"https://api.github.com/users/colinking/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-06-28T21:59:46Z","updated_at":"2017-06-28T22:06:42Z","closed_at":null,"body":"`emitCloudWatchMetrics` in `DynamoDBReplicationEmitter.java` calls `putMetricDataAsync` on the cloudwatch object, which is an instance of `NoopCloudWatch` when `--dontPublishCloudwatch` is specified. This leads to unintended behavior, and metric publishing failures."},"comment":{"url":"https://api.github.com/repos/awslabs/dynamodb-cross-region-library/issues/comments/311805977","html_url":"https://github.com/awslabs/dynamodb-cross-region-library/issues/46#issuecomment-311805977","issue_url":"https://api.github.com/repos/awslabs/dynamodb-cross-region-library/issues/46","id":311805977,"user":{"login":"colinking","id":2907397,"avatar_url":"https://avatars3.githubusercontent.com/u/2907397?v=3","gravatar_id":"","url":"https://api.github.com/users/colinking","html_url":"https://github.com/colinking","followers_url":"https://api.github.com/users/colinking/followers","following_url":"https://api.github.com/users/colinking/following{/other_user}","gists_url":"https://api.github.com/users/colinking/gists{/gist_id}","starred_url":"https://api.github.com/users/colinking/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/colinking/subscriptions","organizations_url":"https://api.github.com/users/colinking/orgs","repos_url":"https://api.github.com/users/colinking/repos","events_url":"https://api.github.com/users/colinking/events{/privacy}","received_events_url":"https://api.github.com/users/colinking/received_events","type":"User","site_admin":false},"created_at":"2017-06-28T22:06:42Z","updated_at":"2017-06-28T22:06:42Z","body":"Actually, upon debugging further, it seems that `DynamoDBMasterToReplicasPipeline` calls the deprecated `DynamoDBReplicationEmitter` constructor which takes only the streams configuration. Therefore, it isn't actually using the `NoopCloudWatch` instance, but instead creates a new Cloudwatch client. Looking further now..."}}
{"actor":{"display_login":"colinking"}}