os/: allow objects with NO_GEN but non-NO_SHARD shard

Signed-off-by: Samuel Just <sam.just@inktank.com>

diff --git a/src/common/hobject.cc b/src/common/hobject.cc
index ffc73aa39e3d7287287f35ce870eb4d274f9aede..ecc8cfde4d715170a6df6c3a06657bf6a343729a 100644 (file)
--- a/src/common/hobject.cc
+++ b/src/common/hobject.cc
@@ -272,7 +272,8 @@ void ghobject_t::generate_test_instances(list<ghobject_t*>& o)
 ostream& operator<<(ostream& out, const ghobject_t& o)
 {
   out << o.hobj;
-  if (o.generation != ghobject_t::NO_GEN) {
+  if (o.generation != ghobject_t::NO_GEN ||
+      o.shard_id != ghobject_t::NO_SHARD) {
     assert(o.shard_id != ghobject_t::NO_SHARD);
     out << "/" << o.generation << "/" << (unsigned)(o.shard_id);
   }

diff --git a/src/os/DBObjectMap.cc b/src/os/DBObjectMap.cc
index f23815a42987c68710b053e37038dc680aadc94b..5334521d3cc5306d78d1139381a8f1b0708b9027 100644 (file)
--- a/src/os/DBObjectMap.cc
+++ b/src/os/DBObjectMap.cc
@@ -156,9 +156,8 @@ string DBObjectMap::ghobject_key(const ghobject_t &oid)
     t += snprintf(t, end - t, ".%llx", (long long unsigned)oid.hobj.pool);
   snprintf(t, end - t, ".%.*X", (int)(sizeof(oid.hobj.hash)*2), oid.hobj.hash);
 
-  if (oid.generation != ghobject_t::NO_GEN) {
-    assert(oid.shard_id != ghobject_t::NO_SHARD);
-
+  if (oid.generation != ghobject_t::NO_GEN ||
+      oid.shard_id != ghobject_t::NO_SHARD) {
     t += snprintf(t, end - t, ".%llx", (long long unsigned)oid.generation);
     t += snprintf(t, end - t, ".%x", (int)oid.shard_id);
   }

diff --git a/src/os/LFNIndex.cc b/src/os/LFNIndex.cc
index 92ccebf024989fd1758784b2d893c1da26fdff1a..0310cb5a6c58bb7ae2f483e2051e42c6b3aa02b9 100644 (file)
--- a/src/os/LFNIndex.cc
+++ b/src/os/LFNIndex.cc
@@ -661,8 +661,8 @@ string LFNIndex::lfn_generate_object_name(const ghobject_t &oid)
     t += snprintf(t, end - t, "%llx", (long long unsigned)oid.hobj.pool);
   full_name += string(buf);
 
-  if (oid.generation != ghobject_t::NO_GEN) {
-    assert(oid.shard_id != ghobject_t::NO_SHARD);
+  if (oid.generation != ghobject_t::NO_GEN ||
+      oid.shard_id != ghobject_t::NO_SHARD) {
     full_name.append("_");
 
     t = buf;