Simplify distribution summary for "days since last seen."
This commit is contained in:
parent
84f85ae098
commit
49c43a6816
|
@ -29,8 +29,7 @@ public class BaseAccountAuthenticator {
|
||||||
private static final String AUTHENTICATION_FAILURE_REASON_TAG_NAME = "reason";
|
private static final String AUTHENTICATION_FAILURE_REASON_TAG_NAME = "reason";
|
||||||
private static final String AUTHENTICATION_ENABLED_REQUIRED_TAG_NAME = "enabledRequired";
|
private static final String AUTHENTICATION_ENABLED_REQUIRED_TAG_NAME = "enabledRequired";
|
||||||
|
|
||||||
private final String daysSinceLastSeenDistributionName = name(getClass(), "authentication", "daysSinceLastSeen");
|
private static final String DAYS_SINCE_LAST_SEEN_DISTRIBUTION_NAME = name(BaseAccountAuthenticator.class, "daysSinceLastSeen");
|
||||||
|
|
||||||
private static final String IS_PRIMARY_DEVICE_TAG = "isPrimary";
|
private static final String IS_PRIMARY_DEVICE_TAG = "isPrimary";
|
||||||
|
|
||||||
private final AccountsManager accountsManager;
|
private final AccountsManager accountsManager;
|
||||||
|
@ -108,10 +107,7 @@ public class BaseAccountAuthenticator {
|
||||||
final long todayInMillisWithOffset = Util.todayInMillisGivenOffsetFromNow(clock, Duration.ofSeconds(lastSeenOffsetSeconds).negated());
|
final long todayInMillisWithOffset = Util.todayInMillisGivenOffsetFromNow(clock, Duration.ofSeconds(lastSeenOffsetSeconds).negated());
|
||||||
|
|
||||||
if (device.getLastSeen() < todayInMillisWithOffset) {
|
if (device.getLastSeen() < todayInMillisWithOffset) {
|
||||||
DistributionSummary.builder(daysSinceLastSeenDistributionName)
|
Metrics.summary(DAYS_SINCE_LAST_SEEN_DISTRIBUTION_NAME, IS_PRIMARY_DEVICE_TAG, String.valueOf(device.isMaster()))
|
||||||
.tags(IS_PRIMARY_DEVICE_TAG, String.valueOf(device.isMaster()))
|
|
||||||
.publishPercentileHistogram()
|
|
||||||
.register(Metrics.globalRegistry)
|
|
||||||
.record(Duration.ofMillis(todayInMillisWithOffset - device.getLastSeen()).toDays());
|
.record(Duration.ofMillis(todayInMillisWithOffset - device.getLastSeen()).toDays());
|
||||||
|
|
||||||
device.setLastSeen(Util.todayInMillis(clock));
|
device.setLastSeen(Util.todayInMillis(clock));
|
||||||
|
|
Loading…
Reference in New Issue