diff --git a/cmds/statsd/src/external/StatsCompanionServicePuller.cpp b/cmds/statsd/src/external/StatsCompanionServicePuller.cpp index 8e96399716794..e1c02d74d6018 100644 --- a/cmds/statsd/src/external/StatsCompanionServicePuller.cpp +++ b/cmds/statsd/src/external/StatsCompanionServicePuller.cpp @@ -45,10 +45,11 @@ bool StatsCompanionServicePuller::Pull(const int tagId, vectorpullData(tagId, &returned_value); if (!status.isOk()) { - ALOGW("error pulling kernel wakelock"); + ALOGW("error pulling for %d", tagId); return false; } data->clear(); + long timestamp = time(nullptr); for (const StatsLogEventWrapper& it : returned_value) { log_msg tmp; tmp.entry_v1.len = it.bytes.size(); @@ -56,9 +57,10 @@ bool StatsCompanionServicePuller::Pull(const int tagId, vectorpush_back(make_shared(tmp)); } - ALOGD("KernelWakelockPuller::pull succeeded!"); + ALOGD("StatsCompanionServicePuller::pull succeeded for %d", tagId); return true; } else { ALOGW("statsCompanion not found!"); diff --git a/cmds/statsd/src/external/StatsPullerManagerImpl.cpp b/cmds/statsd/src/external/StatsPullerManagerImpl.cpp index 3ffcd77bea445..c4688a2b1c3de 100644 --- a/cmds/statsd/src/external/StatsPullerManagerImpl.cpp +++ b/cmds/statsd/src/external/StatsPullerManagerImpl.cpp @@ -76,7 +76,9 @@ bool StatsPullerManagerImpl::Pull(int tagId, vector>* data) if (DEBUG) ALOGD("Initiating pulling %d", tagId); if (mPullers.find(tagId) != mPullers.end()) { - return mPullers.find(tagId)->second->Pull(tagId, data); + bool ret = mPullers.find(tagId)->second->Pull(tagId, data); + ALOGD("pulled %d items", (int)data->size()); + return ret; } else { ALOGD("Unknown tagId %d", tagId); return false; // Return early since we don't know what to pull. diff --git a/cmds/statsd/src/metrics/GaugeMetricProducer.cpp b/cmds/statsd/src/metrics/GaugeMetricProducer.cpp index 2284ff174c350..50cc8d407fb98 100644 --- a/cmds/statsd/src/metrics/GaugeMetricProducer.cpp +++ b/cmds/statsd/src/metrics/GaugeMetricProducer.cpp @@ -99,6 +99,9 @@ GaugeMetricProducer::GaugeMetricProducer(const ConfigKey& key, const GaugeMetric GaugeMetricProducer::~GaugeMetricProducer() { VLOG("~GaugeMetricProducer() called"); + if (mPullTagId != -1) { + mStatsPullerManager.UnRegisterReceiver(mPullTagId, this); + } } void GaugeMetricProducer::startNewProtoOutputStreamLocked(long long startTime) {