From b376458963c7cda75678ca3b23af112d6576d1ca Mon Sep 17 00:00:00 2001 From: Jonathan Klabunde Tomer <125505367+jkt-signal@users.noreply.github.com> Date: Fri, 7 Jun 2024 10:24:42 -0700 Subject: [PATCH] include deleted-messages counter on both message deletion paths --- .../whispersystems/textsecuregcm/storage/MessagesDynamoDb.java | 1 + 1 file changed, 1 insertion(+) diff --git a/service/src/main/java/org/whispersystems/textsecuregcm/storage/MessagesDynamoDb.java b/service/src/main/java/org/whispersystems/textsecuregcm/storage/MessagesDynamoDb.java index 8080a18b1..88663972c 100644 --- a/service/src/main/java/org/whispersystems/textsecuregcm/storage/MessagesDynamoDb.java +++ b/service/src/main/java/org/whispersystems/textsecuregcm/storage/MessagesDynamoDb.java @@ -265,6 +265,7 @@ public class MessagesDynamoDb extends AbstractDynamoDbStore { .thenApplyAsync(deleteItemResponse -> { if (deleteItemResponse.attributes() != null && deleteItemResponse.attributes().containsKey(KEY_PARTITION)) { try { + Metrics.counter(MESSAGES_DELETED_BY_SCHEME_COUNTER_NAME, Tags.of("scheme", scheme.name())).increment(); return Optional.of(convertItemToEnvelope(deleteItemResponse.attributes())); } catch (final InvalidProtocolBufferException e) { logger.error("Failed to parse envelope", e);