Fixes as per review comments

Author: rahuldutta90

src/ResourceManager/DataLakeStore/Commands.DataLakeStore/DataPlaneCommands/RemoveAzureRmDataLakeStoreItemAclEntry.cs

@@ -94,8 +94,7 @@ public class RemoveAzureDataLakeStoreItemAclEntry : DataLakeStoreFileSystemCmdle
         )]
         public int Concurrency { get; set; } = -1;
 
-        [Parameter(ValueFromPipelineByPropertyName = true, Mandatory = false,
-            HelpMessage =
+        [Parameter(Mandatory = false, HelpMessage =
                 "If passed then progress status is showed. Only applicable when recursive Acl remove is done."
         )]
         public SwitchParameter ShowProgress { get; set; }

src/ResourceManager/DataLakeStore/Commands.DataLakeStore/DataPlaneCommands/SetAzureRmDataLakeStoreItemAcl.cs

@@ -65,7 +65,7 @@ public class SetAzureDataLakeStoreItemAcl : DataLakeStoreFileSystemCmdletBase
         )]
         public int Concurrency { get; set; } = -1;
 
-        [Parameter(ValueFromPipelineByPropertyName = true, Mandatory = false,
+        [Parameter(Mandatory = false,
             HelpMessage =
                 "If passed then progress status is showed. Only applicable when recursive Acl Set is done."
         )]

src/ResourceManager/DataLakeStore/Commands.DataLakeStore/DataPlaneCommands/SetAzureRmDataLakeStoreItemAclEntry.cs

@@ -95,7 +95,7 @@ public class SetAzureDataLakeStoreItemAclEntry : DataLakeStoreFileSystemCmdletBa
         )]
         public int Concurrency { get; set; } = -1;
 
-        [Parameter(ValueFromPipelineByPropertyName = true, Mandatory = false,
+        [Parameter(Mandatory = false,
             HelpMessage =
                 "If passed then progress status is showed. Only applicable when recursive Acl modify is done."
         )]
@@ -106,29 +106,18 @@ public override void ExecuteCmdlet()
             WriteWarning(Resources.IncorrectOutputTypeWarning);
             var aclSpec = ParameterSetName.Equals(BaseParameterSetName)
                 ? Acl.Select(entry => entry.ParseDataLakeStoreItemAce()).ToList()
-                : new List<AclEntry>() { new AclEntry((AclType)AceType, Id.ToString(), Default ? AclScope.Default : AclScope.Access, (AclAction)Permissions) };
-
-            int numNonDefaultAcls = 0;
-
-            if (Recurse)
-            {
-                foreach (var aclEntry in aclSpec)
-                {
-                    if (aclEntry.Scope == AclScope.Access)
-                    {
-                        numNonDefaultAcls++;
-                    }
-                }
-            }
+                : new List<AclEntry> { new AclEntry((AclType)AceType, Id.ToString(), Default ? AclScope.Default : AclScope.Access, (AclAction)Permissions) };
 
+            bool recurseAndDefaultAcls = Recurse && !aclSpec.Any(aclEntry => aclEntry.Scope == AclScope.Access);
+            
             // For recurse and all acls are default, give a warning message
-            if (Recurse && numNonDefaultAcls == 0)
+            if (recurseAndDefaultAcls)
             {
                 WriteWarning(Resources.SetOnlyDefaultAclRecursively);
             }
 
             ConfirmAction(
-                string.Format(Resources.SetDataLakeStoreItemAcl, Path.OriginalPath) + (Recurse && numNonDefaultAcls == 0? "\n" + Resources.SetOnlyDefaultAclRecursively : ""),
+                string.Format(Resources.SetDataLakeStoreItemAcl, Path.OriginalPath) + (recurseAndDefaultAcls ? "\n" + Resources.SetOnlyDefaultAclRecursively : ""),
                 Path.OriginalPath,
                 () =>
                 {