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

Commit 8bcd7593 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 am: 35333908 

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

Change-Id: I6309f89afe59ea7ae8dcef88a306d969d8e2b735
parents cbd31021 35333908

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);