sstables: stop including metadata_collector.hh in sstables.hh

metadata collector is rarely used so it's better to include it only
in those few places.

Signed-off-by: Piotr Jastrzebski <piotr@scylladb.com>

api/column_family.cc

@@ -24,6 +24,7 @@
 #include <vector>
 #include <seastar/http/exception.hh>
 #include "sstables/sstables.hh"
+#include "sstables/metadata_collector.hh"
 #include "utils/estimated_histogram.hh"
 #include <algorithm>
 #include "db/system_keyspace_view_types.hh"

sstables/sstables.cc

@@ -43,6 +43,7 @@
 #include "writer.hh"
 #include "m_format_read_helpers.hh"
 #include "sstables.hh"
+#include "metadata_collector.hh"
 #include "progress_monitor.hh"
 #include "compress.hh"
 #include "unimplemented.hh"

sstables/sstables.hh

@@ -43,7 +43,6 @@
 #include "schema_fwd.hh"
 #include "utils/i_filter.hh"
 #include <seastar/core/stream.hh>
-#include "metadata_collector.hh"
 #include "encoding_stats.hh"
 #include "filter.hh"
 #include "exceptions.hh"
@@ -82,6 +81,7 @@ extern logging::logger sstlog;
 class key;
 class sstable_writer;
 class sstables_manager;
+class metadata_collector;
 
 template<typename T>
 concept ConsumeRowsContext =

test/boost/sstable_datafile_test.cc

@@ -35,6 +35,7 @@
 #include "schema_builder.hh"
 #include "database.hh"
 #include "sstables/leveled_manifest.hh"
+#include "sstables/metadata_collector.hh"
 #include <memory>
 #include "test/boost/sstable_test.hh"
 #include <seastar/core/seastar.hh>