Merge "Don't just clear mPastBuckets when onDumpReport is called."

2017-11-18 23:46:35 +00:00
parent 744988cdce 30b6a20c99
commit 43699661dd
1 changed files with 1 additions and 3 deletions
--- a/cmds/statsd/src/metrics/DurationMetricProducer.cpp
+++ b/cmds/statsd/src/metrics/DurationMetricProducer.cpp
@@ -222,10 +222,8 @@ std::unique_ptr<std::vector<uint8_t>> DurationMetricProducer::onDumpReport() {
                  (long long)mCurrentBucketStartTimeNs);

    std::unique_ptr<std::vector<uint8_t>> buffer = serializeProto();
-
    startNewProtoOutputStream(endTime);
-    mPastBuckets.clear();
-
+    // TODO: Properly clear the old buckets.
    return buffer;
 }