Files
scylladb/sstables/random_access_reader.hh
copilot-swe-agent[bot] 168e0a40e3 Fix oversized allocation in sstables::parse by using fragmented buffer
This addresses issue where reading summary positions could cause
large contiguous memory allocations (249856 bytes reported).
Added read_exactly_fragmented() method to random_access_reader to
support reading into fragmented buffers, avoiding oversized allocations.

Co-authored-by: mykaul <4655593+mykaul@users.noreply.github.com>
2025-11-06 11:57:10 +00:00

62 lines
1.5 KiB
C++

/*
* Copyright (C) 2018-present ScyllaDB
*/
/*
* SPDX-License-Identifier: LicenseRef-ScyllaDB-Source-Available-1.0
*/
#pragma once
#include "exceptions.hh"
#include <memory>
#include <seastar/core/file.hh>
#include <seastar/core/fstream.hh>
#include <seastar/core/future.hh>
#include <seastar/core/gate.hh>
#include <seastar/core/iostream.hh>
#include <seastar/core/temporary_buffer.hh>
#include "seastarx.hh"
#include "utils/fragmented_temporary_buffer.hh"
namespace sstables {
class random_access_reader {
std::unique_ptr <input_stream<char>> _in;
protected:
virtual input_stream<char> open_at(uint64_t pos) = 0;
void set(input_stream<char> in) {
parse_assert(!_in);
_in = std::make_unique<input_stream<char>>(std::move(in));
}
public:
future <temporary_buffer<char>> read_exactly(size_t n) noexcept;
future<fragmented_temporary_buffer> read_exactly_fragmented(size_t n) noexcept;
future<> seek(uint64_t pos) noexcept;
bool eof() const noexcept { return _in->eof(); }
virtual future<> close() noexcept;
virtual ~random_access_reader() {}
};
class file_random_access_reader : public random_access_reader {
file _file;
uint64_t _file_size;
size_t _buffer_size;
unsigned _read_ahead;
public:
virtual input_stream<char> open_at(uint64_t pos) override;
explicit file_random_access_reader(file f, uint64_t file_size, size_t buffer_size = 8192, unsigned read_ahead = 4);
virtual future<> close() noexcept override;
};
}