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

Commit 35333908 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Incident dump does not include current bucket" into rvc-dev am: 298dd714 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/11852825

Change-Id: I3529a17b6a8e2bcaeb4fc47ec72053b9761ff6a4
parents b9ed45c3 298dd714

cmds/statsd/src/StatsService.cpp

+4 −1
Original line number Original line Diff line number Diff line
@@ -267,8 +267,11 @@ void StatsService::dumpIncidentSection(int out) { 
    for (const ConfigKey& configKey : mConfigManager->GetAllConfigKeys()) {

    for (const ConfigKey& configKey : mConfigManager->GetAllConfigKeys()) {

        uint64_t reportsListToken =

        uint64_t reportsListToken =

                proto.start(FIELD_TYPE_MESSAGE | FIELD_COUNT_REPEATED | FIELD_ID_REPORTS_LIST);

                proto.start(FIELD_TYPE_MESSAGE | FIELD_COUNT_REPEATED | FIELD_ID_REPORTS_LIST);

        // Don't include the current bucket to avoid skipping buckets.

        // If we need to include the current bucket later, consider changing to NO_TIME_CONSTRAINTS

        // or other alternatives to avoid skipping buckets for pulled metrics.

        mProcessor->onDumpReport(configKey, getElapsedRealtimeNs(),

        mProcessor->onDumpReport(configKey, getElapsedRealtimeNs(),

                                 true /* includeCurrentBucket */, false /* erase_data */,

                                 false /* includeCurrentBucket */, false /* erase_data */,

                                 ADB_DUMP,

                                 ADB_DUMP,

                                 FAST,

                                 FAST,

                                 &proto);

                                 &proto);