From 0d720dfc1727dae23e81fb9c00fee7fd801c28ba Mon Sep 17 00:00:00 2001 From: Herman Lee Date: Wed, 28 Oct 2015 14:07:22 -0700 Subject: [PATCH] Use the correct variable when fetching table properties. Summary: An uninitialized parameter was being passed into the call to fetch the table properties during the compaction notification callbacks. Test Plan: Build it with myrocks and verify unit test passed. Run unit tests. Reviewers: rven, yhchiang, igor Reviewed By: igor Subscribers: dhruba Differential Revision: https://reviews.facebook.net/D49635 --- db/db_impl.cc | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/db/db_impl.cc b/db/db_impl.cc index e0f520dd4..928302411 100644 --- a/db/db_impl.cc +++ b/db/db_impl.cc @@ -1873,8 +1873,7 @@ void DBImpl::NotifyOnCompactionCompleted( info.input_files.push_back(fn); if (info.table_properties.count(fn) == 0) { std::shared_ptr tp; - std::string fname; - auto s = cfd->current()->GetTableProperties(&tp, fmd, &fname); + auto s = cfd->current()->GetTableProperties(&tp, fmd, &fn); if (s.ok()) { info.table_properties[fn] = tp; }