From 69b2ed7939b88cf76d252c8c8122f0f52f5f2751 Mon Sep 17 00:00:00 2001 From: Howard Ro Date: Fri, 14 Feb 2020 13:32:42 -0800 Subject: Limit metricslogger call into statsdw for events Histogram and counters are not in the scope of the migration into statsd. Therefore, we can optimize the statsd socket by not propagating histogram and counter based metricslogger calls. Test: compiles & boots Change-Id: I1fe277bcef438c6aceb2b674d6aec5fe0f48a270 --- core/java/com/android/internal/logging/MetricsLogger.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/core/java/com/android/internal/logging/MetricsLogger.java b/core/java/com/android/internal/logging/MetricsLogger.java index 140c410e8de6..75eb4aa052b0 100644 --- a/core/java/com/android/internal/logging/MetricsLogger.java +++ b/core/java/com/android/internal/logging/MetricsLogger.java @@ -55,8 +55,11 @@ public class MetricsLogger { protected void saveLog(LogMaker log) { // TODO(b/116684537): Flag guard logging to event log and statsd socket. EventLogTags.writeSysuiMultiAction(log.serialize()); - FrameworkStatsLog.write(FrameworkStatsLog.KEY_VALUE_PAIRS_ATOM, - /* UID is retrieved from statsd side */ 0, log.getEntries()); + if (log.getCategory() != MetricsEvent.RESERVED_FOR_LOGBUILDER_COUNTER + && log.getCategory() != MetricsEvent.RESERVED_FOR_LOGBUILDER_HISTOGRAM) { + FrameworkStatsLog.write(FrameworkStatsLog.KEY_VALUE_PAIRS_ATOM, + /* UID is retrieved from statsd side */ 0, log.getEntries()); + } } public static final int VIEW_UNKNOWN = MetricsEvent.VIEW_UNKNOWN; -- cgit v1.2.3-59-g8ed1b