fix little

Author: lxy-9602

include/paimon/global_index/global_index_io_meta.h

@@ -28,10 +28,6 @@
 namespace paimon {
 /// Metadata describing a single file entry in a global index.
 struct PAIMON_EXPORT GlobalIndexIOMeta {
-    GlobalIndexIOMeta(const std::string& _file_path, int64_t _file_size,
-                      const std::shared_ptr<Bytes>& _metadata)
-        : GlobalIndexIOMeta(_file_path, _file_size, _metadata, std::nullopt) {}
-
     GlobalIndexIOMeta(const std::string& _file_path, int64_t _file_size,
                       const std::shared_ptr<Bytes>& _metadata,
                       const std::optional<std::vector<int32_t>>& _extra_field_ids)

src/paimon/common/global_index/btree/btree_compatibility_test.cpp

@@ -115,7 +115,8 @@ class BTreeCompatibilityTest : public ::testing::Test {
         PAIMON_ASSIGN_OR_RAISE(auto file_status, fs_->GetFileStatus(bin_path));
         auto file_size = file_status->GetLen();
 
-        GlobalIndexIOMeta io_meta(bin_path, file_size, meta_bytes);
+        GlobalIndexIOMeta io_meta(bin_path, file_size, meta_bytes,
+                                  /*extra_field_ids=*/std::nullopt);
         std::vector<GlobalIndexIOMeta> metas = {io_meta};
 
         auto schema = arrow::schema({arrow::field("testField", arrow_type)});

src/paimon/common/global_index/btree/btree_file_meta_selector_test.cpp

@@ -48,12 +48,12 @@ class BTreeFileMetaSelectorTest : public ::testing::Test {
         // file6: only-nulls file (no keys, has_nulls=true)
         auto meta6 = std::make_shared<BTreeIndexMeta>(nullptr, nullptr, true);
         files_ = {
-            GlobalIndexIOMeta("file1", 1, meta1->Serialize(pool_.get())),
-            GlobalIndexIOMeta("file2", 1, meta2->Serialize(pool_.get())),
-            GlobalIndexIOMeta("file3", 1, meta3->Serialize(pool_.get())),
-            GlobalIndexIOMeta("file4", 1, meta4->Serialize(pool_.get())),
-            GlobalIndexIOMeta("file5", 1, meta5->Serialize(pool_.get())),
-            GlobalIndexIOMeta("file6", 1, meta6->Serialize(pool_.get())),
+            GlobalIndexIOMeta("file1", 1, meta1->Serialize(pool_.get()), std::nullopt),
+            GlobalIndexIOMeta("file2", 1, meta2->Serialize(pool_.get()), std::nullopt),
+            GlobalIndexIOMeta("file3", 1, meta3->Serialize(pool_.get()), std::nullopt),
+            GlobalIndexIOMeta("file4", 1, meta4->Serialize(pool_.get()), std::nullopt),
+            GlobalIndexIOMeta("file5", 1, meta5->Serialize(pool_.get()), std::nullopt),
+            GlobalIndexIOMeta("file6", 1, meta6->Serialize(pool_.get()), std::nullopt),
         };
     }
 

src/paimon/common/global_index/btree/btree_global_index_integration_test.cpp

@@ -1703,7 +1703,7 @@ TEST_P(BTreeGlobalIndexIntegrationTest, CreateReaderWithMultiFieldSchema) {
                                                   {BtreeDefs::kBtreeIndexCompression, GetParam()}};
     ASSERT_OK_AND_ASSIGN(auto indexer, BTreeGlobalIndexer::Create(options));
 
-    GlobalIndexIOMeta meta("fake_path", 100, nullptr);
+    GlobalIndexIOMeta meta("fake_path", 100, nullptr, /*extra_field_ids=*/std::nullopt);
     std::vector<GlobalIndexIOMeta> metas = {meta};
 
     ASSERT_NOK_WITH_MSG(indexer->CreateReader(c_schema.get(), file_reader, metas, pool_),

src/paimon/common/global_index/btree/btree_global_index_writer.cpp

@@ -199,7 +199,8 @@ Result<std::vector<GlobalIndexIOMeta>> BTreeGlobalIndexWriter::Finish() {
     // Create GlobalIndexIOMeta
     std::string file_path = file_writer_->ToPath(index_file_name_);
     PAIMON_ASSIGN_OR_RAISE(int64_t file_size, file_writer_->GetFileSize(index_file_name_));
-    GlobalIndexIOMeta io_meta(file_path, file_size, meta_bytes);
+    GlobalIndexIOMeta io_meta(file_path, file_size, meta_bytes,
+                              /*extra_field_ids=*/std::nullopt);
     return std::vector<GlobalIndexIOMeta>{io_meta};
 }
 

src/paimon/common/global_index/wrap/file_index_writer_wrapper.h

@@ -66,7 +66,7 @@ class FileIndexWriterWrapper : public GlobalIndexWriter {
         PAIMON_RETURN_NOT_OK(out->Flush());
         PAIMON_RETURN_NOT_OK(out->Close());
         GlobalIndexIOMeta meta(file_manager_->ToPath(file_name), /*file_size=*/bytes->size(),
-                               /*metadata=*/nullptr);
+                               /*metadata=*/nullptr, /*extra_field_ids=*/std::nullopt);
         return std::vector<GlobalIndexIOMeta>({meta});
     }
 

src/paimon/global_index/lucene/lucene_global_index_writer.cpp

@@ -237,7 +237,7 @@ Result<std::vector<GlobalIndexIOMeta>> LuceneGlobalIndexWriter::Finish() {
     PAIMON_RETURN_NOT_OK(RapidJsonUtil::ToJsonString(options_, &options_json));
     auto meta_bytes = std::make_shared<Bytes>(options_json, pool_.get());
     GlobalIndexIOMeta meta(file_writer_->ToPath(index_file_name), file_size,
-                           /*metadata=*/meta_bytes);
+                           /*metadata=*/meta_bytes, /*extra_field_ids=*/std::nullopt);
     return std::vector<GlobalIndexIOMeta>({meta});
 }
 

src/paimon/global_index/lumina/lumina_global_index.cpp

@@ -895,7 +895,7 @@ Result<std::vector<GlobalIndexIOMeta>> LuminaIndexWriter::Finish() {
     PAIMON_RETURN_NOT_OK(RapidJsonUtil::ToJsonString(lumina_options_, &options_json));
     auto meta_bytes = std::make_shared<Bytes>(options_json, pool_->GetPaimonPool().get());
     GlobalIndexIOMeta meta(file_manager_->ToPath(index_file_name), file_size,
-                           /*metadata=*/meta_bytes);
+                           /*metadata=*/meta_bytes, /*extra_field_ids=*/std::nullopt);
     return std::vector<GlobalIndexIOMeta>({meta});
 }
 

src/paimon/global_index/tantivy/tantivy_equivalence_test.cpp

@@ -354,7 +354,7 @@ TEST_F(TantivyEquivalenceTest, BenchmarkBuildAndQuery) {
     // -------- Lucene: write + open + queries --------
     auto lroot = paimon::test::UniqueTestDirectory::Create();
     std::map<std::string, std::string> lopt = {{"lucene-fts.write.tmp.directory", lroot->Str()}};
-    GlobalIndexIOMeta lmeta{"", 0, nullptr};
+    GlobalIndexIOMeta lmeta{"", 0, nullptr, std::nullopt};
     auto lwrite_ms =
         time_ms([&] { lmeta = WriteOne("lucene-fts", data_type, lopt, array, lroot->Str()); });
     auto lreader = OpenOne("lucene-fts", data_type, lopt, lmeta, lroot->Str());
@@ -371,7 +371,7 @@ TEST_F(TantivyEquivalenceTest, BenchmarkBuildAndQuery) {
 
     // -------- Tantivy: write + open + queries --------
     auto troot = paimon::test::UniqueTestDirectory::Create();
-    GlobalIndexIOMeta tmeta{"", 0, nullptr};
+    GlobalIndexIOMeta tmeta{"", 0, nullptr, std::nullopt};
     auto twrite_ms =
         time_ms([&] { tmeta = WriteOne("tantivy-fulltext", data_type, {}, array, troot->Str()); });
     auto treader = OpenOne("tantivy-fulltext", data_type, {}, tmeta, troot->Str());

src/paimon/global_index/tantivy/tantivy_global_index_writer.cpp

@@ -162,7 +162,7 @@ Result<std::vector<GlobalIndexIOMeta>> TantivyGlobalIndexWriter::Finish() {
     PAIMON_RETURN_NOT_OK(RapidJsonUtil::ToJsonString(options_, &options_json));
     auto meta_bytes = std::make_shared<Bytes>(options_json, pool_.get());
     GlobalIndexIOMeta meta(file_writer_->ToPath(index_file_name), file_size,
-                           /*metadata=*/meta_bytes);
+                           /*metadata=*/meta_bytes, /*extra_field_ids=*/std::nullopt);
     return std::vector<GlobalIndexIOMeta>({meta});
 }