Merge rebase

man-group · Feb 7, 2025 · 46e1d3a · 46e1d3a
1 parent 359ea68
commit 46e1d3a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/cpp/arcticdb/storage/s3/s3_client_wrapper.cpp b/cpp/arcticdb/storage/s3/s3_client_wrapper.cpp
@@ -102,15 +102,15 @@ folly::Future<S3Result<Segment>> S3ClientTestWrapper::get_object_async(
 
 S3Result<std::monostate> S3ClientTestWrapper::put_object(
         const std::string &s3_object_name,
-        Segment &&segment,
+        Segment &segment,
         const std::string &bucket_name,
         PutHeader header) {
     auto maybe_error = has_failure_trigger(bucket_name);
     if (maybe_error.has_value()) {
         return {*maybe_error};
     }
 
-    return actual_client_->put_object(s3_object_name, std::move(segment), bucket_name, header);
+    return actual_client_->put_object(s3_object_name, segment, bucket_name, header);
 }
 
 S3Result<DeleteOutput> S3ClientTestWrapper::delete_objects(

diff --git a/cpp/arcticdb/storage/s3/s3_client_wrapper.hpp b/cpp/arcticdb/storage/s3/s3_client_wrapper.hpp
@@ -50,7 +50,7 @@ class S3ClientTestWrapper : public S3ClientInterface {
 
     S3Result<std::monostate> put_object(
         const std::string& s3_object_name,
-        Segment&& segment,
+        Segment& segment,
         const std::string& bucket_name,
         PutHeader header = PutHeader::NONE) override;