metrics.cc: Do not merge empty histogram

It is common to have empty histograms. This patch adds an optimization when merging an empty histogram to another histogram to return early. Signed-off-by: Amnon Heiman <amnon@scylladb.com> Closes scylladb#1294
travisdowns · Nov 14, 2022 · f8eb364 · f8eb364
1 parent 6614ac5
commit f8eb364
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/src/core/metrics.cc b/src/core/metrics.cc
@@ -376,6 +376,9 @@ const bool metric_disabled = false;
 
 
 histogram& histogram::operator+=(const histogram& c) {
+    if (c.sample_count == 0) {
+        return *this;
+    }
     for (size_t i = 0; i < c.buckets.size(); i++) {
         if (buckets.size() <= i) {
             buckets.push_back(c.buckets[i]);