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

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

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

Change-Id: Ia469e2df4645ae3d5b59fe91d4dcd0d56e4cad8a
parents b2160819 58a0b9c5

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