From c2eca197374735bc70d003c0ccd71ed4f25fc784 Mon Sep 17 00:00:00 2001 From: Glauber Costa Date: Wed, 5 Aug 2015 20:38:10 -0500 Subject: [PATCH] sstable_test: fix check_toc_func We are currently failing the sstable test. The reason is that we use the store() function for test purposes, and that function does not store the TOC component. It was removed by Aviccident in 3a5e3c88. Because that function is only used for testing purposes, it doesn't need to write the Index and Data components: we can then remove them from the list. Signed-off-by: Glauber Costa --- sstables/sstables.cc | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/sstables/sstables.cc b/sstables/sstables.cc index 2d01cbcda5..739619c932 100644 --- a/sstables/sstables.cc +++ b/sstables/sstables.cc @@ -878,12 +878,14 @@ future<> sstable::load() { } future<> sstable::store() { - // TODO: write other components as well. + _components.erase(component_type::Index); + _components.erase(component_type::Data); return seastar::async([this] { write_statistics(); write_compression(); write_filter(); write_summary(); + write_toc(); }); }