Storage/High throughput append blob (Azure#4586)

High Throughput Append Blob
Change HandleItem.accessRights to AccessRights

Author: microzchang

sdk/storage/assets.json

@@ -2,5 +2,5 @@
   "AssetsRepo": "Azure/azure-sdk-assets",
   "AssetsRepoPrefixPath": "cpp",
   "TagPrefix": "cpp/storage",
-  "Tag": "cpp/storage_497b3668b6"
+  "Tag": "cpp/storage_8f2bd0dbe5"
 }

sdk/storage/azure-storage-blobs/inc/azure/storage/blobs/rest_client.hpp

@@ -31,7 +31,7 @@ namespace Azure { namespace Storage { namespace Blobs {
     /**
      * The version used for the operations to Azure storage services.
      */
-    constexpr static const char* ApiVersion = "2021-12-02";
+    constexpr static const char* ApiVersion = "2022-11-02";
   } // namespace _detail
   namespace Models {
     /**

sdk/storage/azure-storage-blobs/src/rest_client.cpp

@@ -102,9 +102,13 @@ directive:
           "name": "ApiVersion",
           "modelAsString": false
           },
-        "enum": ["2021-12-02"],
+        "enum": ["2022-11-02"],
         "description": "The version used for the operations to Azure storage services."
       };
+  - from: swagger-document
+    where: $.parameters
+    transform: >
+      $.ApiVersionParameter.enum[0] = "2022-11-02";
 ```
 
 ### Rename Operations

sdk/storage/azure-storage-blobs/swagger/README.md

@@ -338,4 +338,15 @@ namespace Azure { namespace Storage { namespace Test {
         appendBlobClient2.AppendBlockFromUri(appendBlobClient.GetUrl() + GetSas(), options2));
   }
 
+  TEST_F(AppendBlobClientTest, DISABLED_LIVEONLY_HighThroughputAppendBlob)
+  {
+    auto appendBlobClient = m_blobContainerClient->GetAppendBlobClient(RandomString());
+    appendBlobClient.Create();
+    auto blockContent = RandomBuffer(static_cast<size_t>(5_MB));
+    auto blockStream = Azure::Core::IO::MemoryBodyStream(blockContent.data(), blockContent.size());
+    appendBlobClient.AppendBlock(blockStream);
+
+    EXPECT_EQ(ReadBodyStream(appendBlobClient.Download().Value.BodyStream), blockContent);
+  }
+
 }}} // namespace Azure::Storage::Test

sdk/storage/azure-storage-blobs/test/ut/append_blob_client_test.cpp

@@ -291,7 +291,7 @@ namespace Azure { namespace Storage { namespace Files { namespace Shares {
       /**
        * Access rights of the handle.
        */
-      Azure::Nullable<ShareFileHandleAccessRights> accessRights;
+      Azure::Nullable<ShareFileHandleAccessRights> AccessRights;
     };
   } // namespace Models
 

sdk/storage/azure-storage-files-shares/inc/azure/storage/files/shares/share_responses.hpp

@@ -609,7 +609,7 @@ namespace Azure { namespace Storage { namespace Files { namespace Shares {
             accessRights |= Models::ShareFileHandleAccessRights::Delete;
           }
         }
-        directoryHandle.accessRights = std::move(accessRights);
+        directoryHandle.AccessRights = std::move(accessRights);
       }
 
       pagedResponse.DirectoryHandles.push_back(std::move(directoryHandle));

sdk/storage/azure-storage-files-shares/src/share_directory_client.cpp

@@ -715,7 +715,7 @@ namespace Azure { namespace Storage { namespace Files { namespace Shares {
             accessRights |= Models::ShareFileHandleAccessRights::Delete;
           }
         }
-        fileHandle.accessRights = std::move(accessRights);
+        fileHandle.AccessRights = std::move(accessRights);
       }
 
       pagedResponse.FileHandles.push_back(std::move(fileHandle));

sdk/storage/azure-storage-files-shares/src/share_file_client.cpp

@@ -1090,7 +1090,7 @@ namespace Azure { namespace Storage { namespace Test {
         | Files::Shares::Models::ShareFileHandleAccessRights::Write
         | Files::Shares::Models::ShareFileHandleAccessRights::Delete;
     EXPECT_EQ(directoryHandles.size(), 1L);
-    EXPECT_TRUE(directoryHandles[0].accessRights.HasValue());
-    EXPECT_EQ(allAccessRights, directoryHandles[0].accessRights.Value());
+    EXPECT_TRUE(directoryHandles[0].AccessRights.HasValue());
+    EXPECT_EQ(allAccessRights, directoryHandles[0].AccessRights.Value());
   }
 }}} // namespace Azure::Storage::Test

sdk/storage/azure-storage-files-shares/test/ut/share_directory_client_test.cpp

@@ -1429,7 +1429,7 @@ namespace Azure { namespace Storage { namespace Test {
         | Files::Shares::Models::ShareFileHandleAccessRights::Write
         | Files::Shares::Models::ShareFileHandleAccessRights::Delete;
     EXPECT_EQ(fileHandles.size(), 1L);
-    EXPECT_TRUE(fileHandles[0].accessRights.HasValue());
-    EXPECT_EQ(allAccessRights, fileHandles[0].accessRights.Value());
+    EXPECT_TRUE(fileHandles[0].AccessRights.HasValue());
+    EXPECT_EQ(allAccessRights, fileHandles[0].AccessRights.Value());
   }
 }}} // namespace Azure::Storage::Test