From a984b3640e6748316f89ca6f1e99714045071c9c Mon Sep 17 00:00:00 2001 From: Chris Eager Date: Thu, 3 Mar 2022 17:20:55 -0800 Subject: [PATCH] Further refine score distribution summary --- .../textsecuregcm/recaptcha/EnterpriseRecaptchaClient.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/service/src/main/java/org/whispersystems/textsecuregcm/recaptcha/EnterpriseRecaptchaClient.java b/service/src/main/java/org/whispersystems/textsecuregcm/recaptcha/EnterpriseRecaptchaClient.java index 674d31ebe..c1e351f20 100644 --- a/service/src/main/java/org/whispersystems/textsecuregcm/recaptcha/EnterpriseRecaptchaClient.java +++ b/service/src/main/java/org/whispersystems/textsecuregcm/recaptcha/EnterpriseRecaptchaClient.java @@ -105,8 +105,9 @@ public class EnterpriseRecaptchaClient implements RecaptchaClient { final DistributionSummary.Builder distributionSummaryBuilder = DistributionSummary.builder( SCORE_DISTRIBUTION_NAME) - .minimumExpectedValue(0.0d) - .maximumExpectedValue(1.0d) + // score is 0.0…1.0, which doesn’t play well with distribution summary bucketing, so scale to 0…100 + .scale(100) + .maximumExpectedValue(100.0d) .tags("action", String.valueOf(expectedAction)); distributionSummaryBuilder.register(Metrics.globalRegistry).record(score);