From f780db102d9d4b4cbcb20ce0a1e85acc5e9cb93f Mon Sep 17 00:00:00 2001
From: yaojun <940334249@qq.com>
Date: Fri, 20 Feb 2026 11:38:29 +0800
Subject: [PATCH] Fix(C++): use std::move to avoid unnecessary copies

---
 cpp/src/parquet/arrow/reader.cc | 2 +-
 cpp/src/parquet/metadata.cc     | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/cpp/src/parquet/arrow/reader.cc b/cpp/src/parquet/arrow/reader.cc
index a77323d29fa..11c4c408da9 100644
--- a/cpp/src/parquet/arrow/reader.cc
+++ b/cpp/src/parquet/arrow/reader.cc
@@ -976,7 +976,7 @@ Status GetReader(const SchemaField& field, const std::shared_ptr<Field>& arrow_f
       if (!schema_child_type.Equals(reader_child_type)) {
         child_field = child_field->WithType(child_reader->field()->type());
       }
-      child_fields.push_back(child_field);
+      child_fields.push_back(std::move(child_field));
       child_readers.emplace_back(std::move(child_reader));
     }
     if (child_fields.empty()) {
diff --git a/cpp/src/parquet/metadata.cc b/cpp/src/parquet/metadata.cc
index 505ace275b1..7e9b358ce06 100644
--- a/cpp/src/parquet/metadata.cc
+++ b/cpp/src/parquet/metadata.cc
@@ -177,9 +177,9 @@ std::shared_ptr<KeyValueMetadata> FromThriftKeyValueMetadata(const Metadata& sou
     std::vector<std::string> values;
     keys.reserve(source.key_value_metadata.size());
     values.reserve(source.key_value_metadata.size());
-    for (const auto& it : source.key_value_metadata) {
-      keys.push_back(it.key);
-      values.push_back(it.value);
+    for (auto& it : source.key_value_metadata) {
+      keys.push_back(std::move(it.key));
+      values.push_back(std::move(it.value));
     }
     metadata = std::make_shared<KeyValueMetadata>(std::move(keys), std::move(values));
   }