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

Commit 58a0b9c5 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:... 

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

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

Change-Id: I29719af623d52a156865dec5de4aa364ff67c212
parents f04d153e ca376d76

cmds/statsd/src/StatsService.cpp

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

        uint64_t reportsListToken =

                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(),

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

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

                                 ADB_DUMP,

                                 FAST,

                                 &proto);