Fix a bug in PutCloudObject when using transfer manager

Author: Duc Hieu Pham

cloud/aws/aws_s3.cc

@@ -864,9 +864,9 @@ Status S3StorageProvider::DoPutCloudObject(const std::string& local_file,
                                            const std::string& object_path,
                                            uint64_t file_size) {
   if (s3client_->HasTransferManager()) {
-    auto handle =
-        s3client_->UploadFile(ToAwsString(local_file), ToAwsString(bucket_name),
-                              ToAwsString(object_path), file_size);
+    auto handle = s3client_->UploadFile(ToAwsString(bucket_name),
+                                        ToAwsString(object_path),
+                                        ToAwsString(local_file), file_size);
     if (handle->GetStatus() != Aws::Transfer::TransferStatus::COMPLETED) {
       auto error = handle->GetLastError();
       std::string errmsg(error.GetMessage().c_str(), error.GetMessage().size());

cloud/db_cloud_test.cc

@@ -118,6 +118,7 @@ class CloudTest : public testing::Test {
 
   void CreateAwsEnv() {
     CloudEnv* aenv;
+    cloud_env_options_.use_aws_transfer_manager = true;
     ASSERT_OK(CloudEnv::NewAwsEnv(base_env_, cloud_env_options_,
                                   options_.info_log, &aenv));
     // To catch any possible file deletion bugs, we set file deletion delay to