From b103bbf26d2f354881dfd12a11ee2f0acbfcacd7 Mon Sep 17 00:00:00 2001
From: "v-yuzhichen@microsoft.com" <v-yuzhichen@microsoft.com>
Date: Tue, 7 Jun 2022 17:03:19 +0800
Subject: [PATCH 1/2] Fix Az.DataLakeStore XML Comments

---
 .../DataPlaneModels/DataLakeStoreFileSystemClient.cs        | 2 ++
 .../DataPlaneModels/DataLakeStoreFileSystemCmdletBase.cs    | 6 +-----
 .../DataLakeStore/Models/DataLakeStoreFirewallRule.cs       | 3 ---
 .../DataLakeStore/Models/DataLakeStoreTrustedIdProvider.cs  | 3 ---
 4 files changed, 3 insertions(+), 11 deletions(-)

diff --git a/src/DataLakeStore/DataLakeStore/DataPlaneModels/DataLakeStoreFileSystemClient.cs b/src/DataLakeStore/DataLakeStore/DataPlaneModels/DataLakeStoreFileSystemClient.cs
index c6c41bd7050b..57474b306e5d 100644
--- a/src/DataLakeStore/DataLakeStore/DataPlaneModels/DataLakeStoreFileSystemClient.cs
+++ b/src/DataLakeStore/DataLakeStore/DataPlaneModels/DataLakeStoreFileSystemClient.cs
@@ -776,6 +776,7 @@ public void GetFileProperties(string accountName, string path, bool getAclUsage,
         /// <param name="accountName">Account name</param>
         /// <param name="filter">Query to match items in trash</param>
         /// <param name="count">Minimum number of entries to search for</param>
+        /// <param name="cmdlet"></param>
         /// <param name="cmdletCancellationToken">CancellationToken</param>
         public IEnumerable<TrashEntry> EnumerateDeletedItems(string accountName, string filter, int count, Cmdlet cmdlet, CancellationToken cmdletCancellationToken = default(CancellationToken))
         {
@@ -801,6 +802,7 @@ public void GetFileProperties(string accountName, string path, bool getAclUsage,
         /// Restore a stream or directory from trash to user space. This is a synchronous operation.
         /// Not threadsafe when Restore is called for same path from different threads. 
         /// </summary>
+        /// <param name="accountName"></param>
         /// <param name="path">The trash directory path in enumeratedeleteditems response</param>
         /// <param name="destination">Path to where the entry should be restored</param>
         /// <param name="type">Type of the entry which is being restored. "file" or "folder"</param>
diff --git a/src/DataLakeStore/DataLakeStore/DataPlaneModels/DataLakeStoreFileSystemCmdletBase.cs b/src/DataLakeStore/DataLakeStore/DataPlaneModels/DataLakeStoreFileSystemCmdletBase.cs
index 48223a60af3c..73c389c9b364 100644
--- a/src/DataLakeStore/DataLakeStore/DataPlaneModels/DataLakeStoreFileSystemCmdletBase.cs
+++ b/src/DataLakeStore/DataLakeStore/DataPlaneModels/DataLakeStoreFileSystemCmdletBase.cs
@@ -74,11 +74,7 @@ internal static bool UsingByteEncoding(Encoding encoding)
         /// Converts the stream type string into an Encoding
         /// </summary>
         /// <param name="content">The content.</param>
-        /// <param name="type">
-        /// This is a string representation of the encoding. It can be
-        /// "string", "unicode", "bigendianunicode", "ascii", "utf7", or "utf8"
-        /// Note, a ToLowerInvariant is done to the type before comparison is made.
-        /// </param>
+        /// <param name="encoding"></param>
         /// <returns>
         /// The encoding that was represented by the string
         /// </returns>
diff --git a/src/DataLakeStore/DataLakeStore/Models/DataLakeStoreFirewallRule.cs b/src/DataLakeStore/DataLakeStore/Models/DataLakeStoreFirewallRule.cs
index 18787387b825..3838c49d67ce 100644
--- a/src/DataLakeStore/DataLakeStore/Models/DataLakeStoreFirewallRule.cs
+++ b/src/DataLakeStore/DataLakeStore/Models/DataLakeStoreFirewallRule.cs
@@ -30,9 +30,6 @@ public class DataLakeStoreFirewallRule
         /// <summary>
         /// Initializes a new instance of the <see cref="DataLakeStoreItem" /> class.
         /// </summary>
-        /// <param name="property">The property.</param>
-        /// <param name="optionalName">The optional name of the file or folder</param>
-        /// <param name="optionalPath">The optional full path to the file or folder, excluding the file or folder name itself.</param>
         public DataLakeStoreFirewallRule(FirewallRule baseRule)
         {
             Name = baseRule.Name;
diff --git a/src/DataLakeStore/DataLakeStore/Models/DataLakeStoreTrustedIdProvider.cs b/src/DataLakeStore/DataLakeStore/Models/DataLakeStoreTrustedIdProvider.cs
index 0bf7ffb00437..2ef69fa12b6b 100644
--- a/src/DataLakeStore/DataLakeStore/Models/DataLakeStoreTrustedIdProvider.cs
+++ b/src/DataLakeStore/DataLakeStore/Models/DataLakeStoreTrustedIdProvider.cs
@@ -28,9 +28,6 @@ public class DataLakeStoreTrustedIdProvider
         /// <summary>
         /// Initializes a new instance of the <see cref="DataLakeStoreItem" /> class.
         /// </summary>
-        /// <param name="property">The property.</param>
-        /// <param name="optionalName">The optional name of the file or folder</param>
-        /// <param name="optionalPath">The optional full path to the file or folder, excluding the file or folder name itself.</param>
         public DataLakeStoreTrustedIdProvider(TrustedIdProvider baseProvider)
         {
             Name = baseProvider.Name;

From 5dac884fa39cba637b1ebcf42b1de8058f7e29a6 Mon Sep 17 00:00:00 2001
From: Ziyue Zheng <v-ziyzhe@microsoft.com>
Date: Fri, 10 Jun 2022 03:27:34 +0800
Subject: [PATCH 2/2] Fix Az.DataLakeStore XML Comments

---
 .../DataPlaneModels/DataLakeStoreFileSystemClient.cs            | 2 +-
 .../DataLakeStore/Models/DataLakeStoreFirewallRule.cs           | 1 +
 .../DataLakeStore/Models/DataLakeStoreTrustedIdProvider.cs      | 1 +
 3 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/DataLakeStore/DataLakeStore/DataPlaneModels/DataLakeStoreFileSystemClient.cs b/src/DataLakeStore/DataLakeStore/DataPlaneModels/DataLakeStoreFileSystemClient.cs
index 57474b306e5d..a30288f4c669 100644
--- a/src/DataLakeStore/DataLakeStore/DataPlaneModels/DataLakeStoreFileSystemClient.cs
+++ b/src/DataLakeStore/DataLakeStore/DataPlaneModels/DataLakeStoreFileSystemClient.cs
@@ -802,7 +802,7 @@ public void GetFileProperties(string accountName, string path, bool getAclUsage,
         /// Restore a stream or directory from trash to user space. This is a synchronous operation.
         /// Not threadsafe when Restore is called for same path from different threads. 
         /// </summary>
-        /// <param name="accountName"></param>
+        /// <param name="accountName">Account name</param>
         /// <param name="path">The trash directory path in enumeratedeleteditems response</param>
         /// <param name="destination">Path to where the entry should be restored</param>
         /// <param name="type">Type of the entry which is being restored. "file" or "folder"</param>
diff --git a/src/DataLakeStore/DataLakeStore/Models/DataLakeStoreFirewallRule.cs b/src/DataLakeStore/DataLakeStore/Models/DataLakeStoreFirewallRule.cs
index 3838c49d67ce..16b6c0d255cf 100644
--- a/src/DataLakeStore/DataLakeStore/Models/DataLakeStoreFirewallRule.cs
+++ b/src/DataLakeStore/DataLakeStore/Models/DataLakeStoreFirewallRule.cs
@@ -30,6 +30,7 @@ public class DataLakeStoreFirewallRule
         /// <summary>
         /// Initializes a new instance of the <see cref="DataLakeStoreItem" /> class.
         /// </summary>
+        /// <param name="baseRule"></param>
         public DataLakeStoreFirewallRule(FirewallRule baseRule)
         {
             Name = baseRule.Name;
diff --git a/src/DataLakeStore/DataLakeStore/Models/DataLakeStoreTrustedIdProvider.cs b/src/DataLakeStore/DataLakeStore/Models/DataLakeStoreTrustedIdProvider.cs
index 2ef69fa12b6b..85939ce4c9a9 100644
--- a/src/DataLakeStore/DataLakeStore/Models/DataLakeStoreTrustedIdProvider.cs
+++ b/src/DataLakeStore/DataLakeStore/Models/DataLakeStoreTrustedIdProvider.cs
@@ -28,6 +28,7 @@ public class DataLakeStoreTrustedIdProvider
         /// <summary>
         /// Initializes a new instance of the <see cref="DataLakeStoreItem" /> class.
         /// </summary>
+        /// <param name="baseProvider"></param>
         public DataLakeStoreTrustedIdProvider(TrustedIdProvider baseProvider)
         {
             Name = baseProvider.Name;