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

Merge "Skip writing metrics to disk if it's entirely composed of no_report_metric" into pi-dev

Browse Source
This commit is contained in:
TreeHugger Robot
2018-05-10 02:02:37 +00:00
committed by Android (Google) Code Review
parent 7c3c4d38a9 028091cb15
commit 3f57b16deb
2 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
cmds/statsd/src/StatsLogProcessor.cpp
View File

@@ -503,7 +503,8 @@ void StatsLogProcessor::flushIfNecessaryLocked(
void StatsLogProcessor::WriteDataToDiskLocked(const ConfigKey& key, void StatsLogProcessor::WriteDataToDiskLocked(const ConfigKey& key,
const int64_t timestampNs, const int64_t timestampNs,
const DumpReportReason dumpReportReason) { const DumpReportReason dumpReportReason) {
if (mMetricsManagers.find(key) == mMetricsManagers.end()) { if (mMetricsManagers.find(key) == mMetricsManagers.end() ||
!mMetricsManagers.find(key)->second->shouldWriteToDisk()) {
return; return;
} }
ProtoOutputStream proto; ProtoOutputStream proto;

4
cmds/statsd/src/metrics/MetricsManager.h
View File

@@ -67,6 +67,10 @@ public:
return !mAllowedPkg.empty(); return !mAllowedPkg.empty();
} }
bool shouldWriteToDisk() const {
return mNoReportMetricIds.size() != mAllMetricProducers.size();
}
void dumpStates(FILE* out, bool verbose); void dumpStates(FILE* out, bool verbose);
inline bool isInTtl(const int64_t timestampNs) const { inline bool isInTtl(const int64_t timestampNs) const {