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

Commit b2532daf authored by Yangster-mac's avatar Yangster-mac Committed by Yang Lu
Browse files

Fix the gauge metric data size estimation. 

Test: statsd test
BUG: b/77906846
Change-Id: Ie5ab54771acadde1e825289fe5ec491cc2225458
parent 68986635

cmds/statsd/src/metrics/GaugeMetricProducer.cpp

+8 −2
Original line number Diff line number Diff line
@@ -186,7 +186,6 @@ void GaugeMetricProducer::onDumpReportLocked(const int64_t dumpTimeNs, 
        flushIfNeededLocked(dumpTimeNs);

    }



    flushIfNeededLocked(dumpTimeNs);

    if (mPastBuckets.empty()) {

        return;

    }
@@ -538,7 +537,14 @@ void GaugeMetricProducer::flushCurrentBucketLocked(const int64_t& eventTimeNs) { 
size_t GaugeMetricProducer::byteSizeLocked() const {

    size_t totalSize = 0;

    for (const auto& pair : mPastBuckets) {

        totalSize += pair.second.size() * kBucketSize;

        for (const auto& bucket : pair.second) {

            totalSize += bucket.mGaugeAtoms.size() * sizeof(GaugeAtom);

            for (const auto& atom : bucket.mGaugeAtoms) {

                if (atom.mFields != nullptr) {

                    totalSize += atom.mFields->size() * sizeof(FieldValue);

                }

            }

        }

    }

    return totalSize;

}