Skip to content

Commit

Permalink
Simplify Table::Open() flow and remove a delete call.
Browse files Browse the repository at this point in the history
-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=167303843
  • Loading branch information
pwnall committed Sep 1, 2017
1 parent 09a3c8e commit 141e767
Showing 1 changed file with 3 additions and 8 deletions.
11 changes: 3 additions & 8 deletions table/table.cc
Expand Up @@ -55,22 +55,19 @@ Status Table::Open(const Options& options,
if (!s.ok()) return s;

// Read the index block
BlockContents contents;
Block* index_block = NULL;
BlockContents index_block_contents;
if (s.ok()) {
ReadOptions opt;
if (options.paranoid_checks) {
opt.verify_checksums = true;
}
s = ReadBlock(file, opt, footer.index_handle(), &contents);
if (s.ok()) {
index_block = new Block(contents);
}
s = ReadBlock(file, opt, footer.index_handle(), &index_block_contents);
}

if (s.ok()) {
// We've successfully read the footer and the index block: we're
// ready to serve requests.
Block* index_block = new Block(index_block_contents);
Rep* rep = new Table::Rep;
rep->options = options;
rep->file = file;
Expand All @@ -81,8 +78,6 @@ Status Table::Open(const Options& options,
rep->filter = NULL;
*table = new Table(rep);
(*table)->ReadMeta(footer);
} else {
delete index_block;
}

return s;
Expand Down

0 comments on commit 141e767

Please sign in to comment.