Further refine score distribution summary
This commit is contained in:
parent
f9c1e411aa
commit
a984b3640e
|
@ -105,8 +105,9 @@ public class EnterpriseRecaptchaClient implements RecaptchaClient {
|
||||||
|
|
||||||
final DistributionSummary.Builder distributionSummaryBuilder = DistributionSummary.builder(
|
final DistributionSummary.Builder distributionSummaryBuilder = DistributionSummary.builder(
|
||||||
SCORE_DISTRIBUTION_NAME)
|
SCORE_DISTRIBUTION_NAME)
|
||||||
.minimumExpectedValue(0.0d)
|
// score is 0.0…1.0, which doesn’t play well with distribution summary bucketing, so scale to 0…100
|
||||||
.maximumExpectedValue(1.0d)
|
.scale(100)
|
||||||
|
.maximumExpectedValue(100.0d)
|
||||||
.tags("action", String.valueOf(expectedAction));
|
.tags("action", String.valueOf(expectedAction));
|
||||||
|
|
||||||
distributionSummaryBuilder.register(Metrics.globalRegistry).record(score);
|
distributionSummaryBuilder.register(Metrics.globalRegistry).record(score);
|
||||||
|
|
Loading…
Reference in New Issue