Possibility of declaring RandomAccessFile::GetStream as a virtual function?

[yukkit]

Describe the usage question you have. Please include as many useful details as possible.

Hello,

I would like to propose an enhancement for the Arrow project.
Specifically, I would like to discuss the possibility of declaring the method RandomAccessFile::GetStream as a virtual function.

Currently, the method can be found at

arrow/cpp/src/arrow/io/interfaces.h

Lines 259 to 260 in 4841cda

	static Result<std::shared_ptr<InputStream>> GetStream(
	std::shared_ptr<RandomAccessFile> file, int64_t file_offset, int64_t nbytes);

I suggest declaring this method as a virtual function and providing the implementation

arrow/cpp/src/arrow/io/interfaces.cc

Lines 267 to 276 in 0b9f817

	Result<std::shared_ptr<InputStream>> RandomAccessFile::GetStream(
	std::shared_ptr<RandomAccessFile> file, int64_t file_offset, int64_t nbytes) {
	if (file_offset < 0) {
	return Status::Invalid("file_offset should be a positive value, got: ", file_offset);
	}
	if (nbytes < 0) {
	return Status::Invalid("nbytes should be a positive value, got: ", nbytes);
	}
	return std::make_shared<FileSegmentReader>(std::move(file), file_offset, nbytes);
	}

as the default implementation.
This would allow users to override the GetStream method by inheriting from the RandomAccessFile class and providing their own implementation to return a custom InputStream.

We are currently using Arrow C++ in our project, and it would greatly benefit us if we could receive our own implementation of InputStream from the GetStream method.

Component(s)

C++