refactor: remove unused path-based reader builder

Author: zjw1111

include/paimon/format/reader_builder.h

@@ -19,7 +19,6 @@
 #pragma once
 
 #include <memory>
-#include <string>
 
 #include "paimon/memory/memory_pool.h"
 #include "paimon/reader/file_batch_reader.h"
@@ -38,9 +37,6 @@ class PAIMON_EXPORT ReaderBuilder {
     /// Build a file batch reader based on the created `InputStream`.
     virtual Result<std::unique_ptr<FileBatchReader>> Build(
         const std::shared_ptr<InputStream>& path) const = 0;
-
-    /// Build a file batch reader based on the file path.
-    virtual Result<std::unique_ptr<FileBatchReader>> Build(const std::string& path) const = 0;
 };
 
 }  // namespace paimon

src/paimon/common/reader/prefetch_file_batch_reader_impl_test.cpp

@@ -100,10 +100,6 @@ class ControlledMockFormatReaderBuilder : public ReaderBuilder {
             data_, schema_, read_batch_size_, read_ranges_, need_prefetch_, set_read_ranges_status);
     }
 
-    Result<std::unique_ptr<FileBatchReader>> Build(const std::string& path) const override {
-        return Status::Invalid("do not support build reader with path in mock format");
-    }
-
  private:
     std::shared_ptr<arrow::Array> data_;
     std::shared_ptr<arrow::DataType> schema_;

src/paimon/format/avro/avro_reader_builder.h

@@ -47,10 +47,6 @@ class AvroReaderBuilder : public ReaderBuilder {
         return AvroFileBatchReader::Create(path, batch_size_, pool_);
     }
 
-    Result<std::unique_ptr<FileBatchReader>> Build(const std::string& path) const override {
-        return Status::Invalid("do not support build reader with path in avro format");
-    }
-
  private:
     const int32_t batch_size_;
     std::shared_ptr<MemoryPool> pool_;

src/paimon/format/blob/blob_reader_builder.h

@@ -48,10 +48,6 @@ class BlobReaderBuilder : public ReaderBuilder {
         return BlobFileBatchReader::Create(input_stream, batch_size_, blob_as_descriptor, pool_);
     }
 
-    Result<std::unique_ptr<FileBatchReader>> Build(const std::string& path) const override {
-        return Status::Invalid("do not support build reader with path in blob format");
-    }
-
  private:
     int32_t batch_size_;
     std::shared_ptr<MemoryPool> pool_;

src/paimon/format/orc/orc_reader_builder.h

@@ -54,10 +54,6 @@ class OrcReaderBuilder : public ReaderBuilder {
         return OrcFileBatchReader::Create(std::move(input_stream), pool_, options_, batch_size_);
     }
 
-    Result<std::unique_ptr<FileBatchReader>> Build(const std::string& path) const override {
-        return Status::Invalid("do not support build reader with path in orc format");
-    }
-
  private:
     int32_t batch_size_ = -1;
     std::shared_ptr<MemoryPool> pool_;

src/paimon/format/parquet/parquet_reader_builder.h

@@ -53,10 +53,6 @@ class ParquetReaderBuilder : public ReaderBuilder {
                                               batch_size_);
     }
 
-    Result<std::unique_ptr<FileBatchReader>> Build(const std::string& path) const override {
-        return Status::Invalid("do not support build reader with path in parquet format");
-    }
-
  private:
     int32_t batch_size_ = -1;
     std::shared_ptr<MemoryPool> pool_;

src/paimon/testing/mock/mock_format_reader_builder.h

@@ -54,10 +54,6 @@ class MockFormatReaderBuilder : public ReaderBuilder {
         return std::make_unique<MockFileBatchReader>(data_, schema_, read_batch_size_);
     }
 
-    Result<std::unique_ptr<FileBatchReader>> Build(const std::string& path) const override {
-        return Status::Invalid("do not support build reader with path in mock format");
-    }
-
  private:
     std::shared_ptr<arrow::Array> data_;
     std::shared_ptr<arrow::DataType> schema_;