From 47316c2d084a11790f50a1f5a88b306c2928e3c5 Mon Sep 17 00:00:00 2001 From: sdong Date: Mon, 3 Aug 2015 11:01:24 -0700 Subject: [PATCH] dump_manifest supports DB with more number of levels Summary: Now ldb dump_manifest refuses to work if there are 20 levels. Extend the limit to 64. Test Plan: Run the tool with 20 number of levels Reviewers: kradhakrishnan, anthony, IslamAbdelRahman, yhchiang Reviewed By: yhchiang Subscribers: leveldb, dhruba Differential Revision: https://reviews.facebook.net/D42879 --- util/ldb_cmd.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/util/ldb_cmd.cc b/util/ldb_cmd.cc index 406b6b999..40f5ae74e 100644 --- a/util/ldb_cmd.cc +++ b/util/ldb_cmd.cc @@ -537,6 +537,7 @@ void DumpManifestFile(std::string file, bool verbose, bool hex, bool json) { // if VersionSet::DumpManifest() depends on any option done by // SanitizeOptions(), we need to initialize it manually. options.db_paths.emplace_back("dummy", 0); + options.num_levels = 64; WriteController wc(options.delayed_write_rate); WriteBuffer wb(options.db_write_buffer_size); VersionSet versions(dbname, &options, sopt, tc.get(), &wb, &wc);