osd/scrub: change missing_digest fix-list type

...allowing for efficient creation and, more important,
direct consumption by the function performing the
actual I/O.

Signed-off-by: Ronen Friedman <rfriedma@redhat.com>

1 files changed, 4 insertions, 3 deletions

diff --git a/src/osd/scrubber/PrimaryLogScrub.cc b/src/osd/scrubber/PrimaryLogScrub.cc
index 53247cae4c5..127f9fab56b 100644
--- a/src/osd/scrubber/PrimaryLogScrub.cc
+++ b/src/osd/scrubber/PrimaryLogScrub.cc
@@ -42,12 +42,12 @@ bool PrimaryLogScrub::get_store_errors(const scrub_ls_arg_t& arg,
   return true;
 }
 
-//  forwarders used by the scrubber backend
-
 /// \todo combine the multiple transactions into a single one
 void PrimaryLogScrub::submit_digest_fixes(const digests_fixes_t& fixes)
 {
-  num_digest_updates_pending += fixes.size();
+  // note: the following line was modified from '+=' to '=', as we should not
+  // encounter previous-chunk digest updates after starting a new chunk
+  num_digest_updates_pending = fixes.size();
   dout(10) << __func__
            << ": num_digest_updates_pending: " << num_digest_updates_pending
            << dendl;
@@ -99,6 +99,7 @@ void PrimaryLogScrub::submit_digest_fixes(const digests_fixes_t& fixes)
   }
 }
 
+//  a forwarder used by the scrubber backend
 
 void PrimaryLogScrub::add_to_stats(const object_stat_sum_t& stat)
 {