Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 94f2fe22 authored by Jack Yu's avatar Jack Yu Committed by android-build-merger
Browse files

Added todo items for a potential metrics issue

am: dc5a63bd

Change-Id: Ibf846f026d137db2c10fd076a408f2702cab5eaf
parents ed7bd6c3 dc5a63bd
Loading
Loading
Loading
Loading
+6 −0
Original line number Diff line number Diff line
@@ -109,6 +109,12 @@ public class ConnectivityMetricsLogger {
            // Log number of skipped events
            Bundle b = new Bundle();
            b.putInt(DATA_KEY_EVENTS_COUNT, mNumSkippedEvents);

            // Log the skipped event.
            // TODO: Note that some of the clients push all states events into the server,
            // If we lose some states logged here, we might mess up the statistics happened at the
            // backend. One of the options is to introduce a non-skippable flag for important events
            // that are logged.
            skippedEventsEvent = new ConnectivityMetricsEvent(mServiceUnblockedTimestampMillis,
                    componentTag, TAG_SKIPPED_EVENTS, b);