From 75ffcefd9036bb3b6c4daed9824c1d577e9cc7e5 Mon Sep 17 00:00:00 2001 From: Glauber Costa Date: Fri, 17 Jul 2015 19:46:38 -0400 Subject: [PATCH] schema: remove set_compressor_params use the builder instead. Sole current user is patched Signed-off-by: Glauber Costa --- schema.hh | 3 --- tests/urchin/sstable_datafile_test.cc | 7 ++++--- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/schema.hh b/schema.hh index 12011456f3..0fca912167 100644 --- a/schema.hh +++ b/schema.hh @@ -231,9 +231,6 @@ public: return _raw._bloom_filter_fp_chance; } sstring thrift_key_validator() const; - void set_compressor_params(compression_parameters c) { - _raw._compressor_params = c; - } const compression_parameters& get_compressor_params() const { return _raw._compressor_params; } diff --git a/tests/urchin/sstable_datafile_test.cc b/tests/urchin/sstable_datafile_test.cc index bf99258e06..b84c19ee63 100644 --- a/tests/urchin/sstable_datafile_test.cc +++ b/tests/urchin/sstable_datafile_test.cc @@ -11,6 +11,7 @@ #include "sstables/compaction.hh" #include "tests/test-utils.hh" #include "schema.hh" +#include "schema_builder.hh" #include "database.hh" #include #include "sstable_test.hh" @@ -883,10 +884,10 @@ SEASTAR_TEST_CASE(datafile_generation_12) { static future<> sstable_compression_test(compressor c, unsigned generation) { return test_setup::do_with_test_directory([c, generation] { - auto& cs = *complex_schema(); - auto s = make_lw_shared(schema(cs)); // NOTE: set a given compressor algorithm to schema. - s->set_compressor_params(c); + schema_builder builder(complex_schema()); + builder.set_compressor_params(c); + auto s = builder.build(); auto mtp = make_lw_shared(s);