From 23119ffacc0456da3cb23ee63f698a3241d0c3a9 Mon Sep 17 00:00:00 2001 From: Sage Weil Date: Thu, 13 Aug 2015 14:37:32 -0400 Subject: [PATCH] Revert "osd/HitSet: make subclasses dump deterministically" This reverts commit 34c048b3cb2b16658e692ac5110115612031510d. --- src/osd/HitSet.h | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/src/osd/HitSet.h b/src/osd/HitSet.h index 51a396a32611a..2b981a3eaacf4 100644 --- a/src/osd/HitSet.h +++ b/src/osd/HitSet.h @@ -237,10 +237,7 @@ public: void dump(Formatter *f) const { f->dump_unsigned("insert_count", count); f->open_array_section("hash_set"); - // dump deterministically - std::set tmp; - tmp.insert(hits.begin(), hits.end()); - for (std::set::iterator p = tmp.begin(); p != tmp.end(); ++p) + for (ceph::unordered_set::const_iterator p = hits.begin(); p != hits.end(); ++p) f->dump_unsigned("hash", *p); f->close_section(); } @@ -317,10 +314,7 @@ public: void dump(Formatter *f) const { f->dump_unsigned("insert_count", count); f->open_array_section("set"); - // dump deterministically - std::set tmp; - tmp.insert(hits.begin(), hits.end()); - for (std::set::iterator p = tmp.begin(); p != tmp.end(); ++p) { + for (ceph::unordered_set::const_iterator p = hits.begin(); p != hits.end(); ++p) { f->open_object_section("object"); p->dump(f); f->close_section(); -- 2.39.5