From 21d24e8ea34dbffa1fcd0d6ea91cdc7b59298fb2 Mon Sep 17 00:00:00 2001 From: Pavel Emelyanov Date: Thu, 18 May 2023 15:57:18 +0300 Subject: [PATCH] index_reader: Move index_reader::get_file_input_stream_options to private: block A "while at it" cleanup. When pathing the method (next patch) it turned out that there are no other callers other than local class, so it _is_ private. Signed-off-by: Pavel Emelyanov --- sstables/index_reader.hh | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/sstables/index_reader.hh b/sstables/index_reader.hh index d88966e412..187c851584 100644 --- a/sstables/index_reader.hh +++ b/sstables/index_reader.hh @@ -752,6 +752,16 @@ private: b.current_list = {}; }); } + + file_input_stream_options get_file_input_stream_options(const io_priority_class& pc) { + file_input_stream_options options; + options.buffer_size = _sstable->sstable_buffer_size; + options.read_ahead = 2; + options.io_priority_class = pc; + options.dynamic_adjustments = _sstable->_index_history; + return options; + } + public: index_reader(shared_sstable sst, reader_permit permit, const io_priority_class& pc, tracing::trace_state_ptr trace_state, use_caching caching, bool single_partition_read = false) @@ -797,15 +807,6 @@ public: return current_partition_entry(_lower_bound); } - file_input_stream_options get_file_input_stream_options(const io_priority_class& pc) { - file_input_stream_options options; - options.buffer_size = _sstable->sstable_buffer_size; - options.read_ahead = 2; - options.io_priority_class = pc; - options.dynamic_adjustments = _sstable->_index_history; - return options; - } - // Returns a pointer to the clustered index cursor for the current partition // or nullptr if there is no clustered index in the current partition. // Returns the same instance until we move to a different partition.