]> git-server-git.apps.pok.os.sepia.ceph.com Git - rocksdb.git/commitdiff
Print unknown writebatch tag (#13062)
authorJay Huh <jewoongh@meta.com>
Thu, 10 Oct 2024 22:34:35 +0000 (15:34 -0700)
committerJay Huh <jewoongh@meta.com>
Thu, 17 Oct 2024 18:01:27 +0000 (11:01 -0700)
Summary:
Add additional info for debugging purpose by doing the same as what WBWI does

https://github.com/facebook/rocksdb/blob/632746bb5b8d9d817b0075b295e1a085e1e543a4/utilities/write_batch_with_index/write_batch_with_index.cc#L274-L276

Pull Request resolved: https://github.com/facebook/rocksdb/pull/13062

Test Plan: CI

Reviewed By: hx235

Differential Revision: D64202297

Pulled By: jaykorean

fbshipit-source-id: 65164fd88420fc72b6db26d1436afe548a653269

db/write_batch.cc

index 3820dccd056bb7ee0bdd4f8582fbe29a02131896..76b59efb68d5e337b218ea27b6a4f8f2b7d35e92 100644 (file)
@@ -502,7 +502,9 @@ Status ReadRecordFromWriteBatch(Slice* input, char* tag,
       break;
     }
     default:
-      return Status::Corruption("unknown WriteBatch tag");
+      return Status::Corruption(
+          "unknown WriteBatch tag",
+          std::to_string(static_cast<unsigned int>(*tag)));
   }
   return Status::OK();
 }
@@ -750,7 +752,9 @@ Status WriteBatchInternal::Iterate(const WriteBatch* wb,
         }
         break;
       default:
-        return Status::Corruption("unknown WriteBatch tag");
+        return Status::Corruption(
+            "unknown WriteBatch tag",
+            std::to_string(static_cast<unsigned int>(tag)));
     }
   }
   if (!s.ok()) {