Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Incident dump does not include current bucket" into rvc-dev

Browse Source
This commit is contained in:
TreeHugger Robot
2020-06-13 02:07:02 +00:00
committed by Android (Google) Code Review
parent 3a8b73a881 61f869e7bb
commit 298dd714eb
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
cmds/statsd/src/StatsService.cpp
View File

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