Skip to content

Commit

Permalink
ldb idump to support non-default column families. (facebook#5594)
Browse files Browse the repository at this point in the history
Summary:
ldb idump now only works for default column family. Extend it.
Pull Request resolved: facebook#5594

Test Plan: Compile and run the tool against a multiple CF DB.

Differential Revision: D16380684

fbshipit-source-id: bfb8af36fdad1806837c90aaaab492d71528aceb
  • Loading branch information
siying authored and facebook-github-bot committed Jul 19, 2019
1 parent abd1fdd commit 6bb3b4b
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion tools/ldb_cmd.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1301,7 +1301,8 @@ void InternalDumpCommand::DoCommand() {

// Cast as DBImpl to get internal iterator
std::vector<KeyVersion> key_versions;
Status st = GetAllKeyVersions(db_, from_, to_, max_keys_, &key_versions);
Status st = GetAllKeyVersions(db_, GetCfHandle(), from_, to_, max_keys_,
&key_versions);
if (!st.ok()) {
exec_state_ = LDBCommandExecuteResult::Failed(st.ToString());
return;
Expand Down

0 comments on commit 6bb3b4b

Please sign in to comment.