Addressing comments.

Author: Milos Todic

src/Sql/Sql/VirtualCluster/Cmdlet/GetAzureSqlVirtualCluster.cs

@@ -54,7 +54,6 @@ public class GetAzureSqlVirtualCluster : VirtualClusterCmdletBase
             HelpMessage = "The name of the resource group.")]
         [Parameter(ParameterSetName = GetByResourceGroupParameterSet,
             Mandatory = false,
-            Position = 1,
             HelpMessage = "The name of the resource group.")]
         [ResourceGroupCompleter]
         [ValidateNotNullOrEmpty]

src/Sql/Sql/VirtualCluster/Cmdlet/RemoveAzureSqlVirtualCluster.cs

@@ -77,7 +77,6 @@ public class RemoveAzureSqlManagedInstance : VirtualClusterCmdletBase
         /// </summary>
         [Parameter(ParameterSetName = RemoveByResourceIdParameterSet,
             Mandatory = true,
-            Position = 0,
             ValueFromPipelineByPropertyName = true,
             HelpMessage = "The resource id of instance object to remove")]
         [ValidateNotNullOrEmpty]

src/Sql/Sql/VirtualCluster/Services/AzureSqlVirtualClusterAdapter.cs

@@ -15,6 +15,7 @@
 using Microsoft.Azure.Commands.Common.Authentication.Abstractions;
 using Microsoft.Azure.Commands.ResourceManager.Common.Tags;
 using Microsoft.Azure.Commands.Sql.VirtualCluster.Model;
+using Microsoft.Azure.Management.Internal.Resources.Utilities.Models;
 using System.Collections.Generic;
 using System.Linq;
 
@@ -99,10 +100,8 @@ private static AzureSqlVirtualClusterModel CreateVirtualClusterModelFromResponse
             AzureSqlVirtualClusterModel virtualCluster = new AzureSqlVirtualClusterModel();
 
             // Extract the resource group name from the ID.
-            // ID is in the form:
-            // /subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/rgName/providers/Microsoft.Sql/virtualClusters/virtualClusterName
-            string[] segments = resp.Id.Split('/');
-            virtualCluster.ResourceGroupName = segments[4];
+            ResourceIdentifier resourceIdentifier = new ResourceIdentifier(resp.Id);
+            virtualCluster.ResourceGroupName = resourceIdentifier.ResourceGroupName;
 
             virtualCluster.VirtualClusterName = resp.Name;
             virtualCluster.Id = resp.Id;

src/Sql/Sql/help/Remove-AzSqlVirtualCluster.md

@@ -26,7 +26,7 @@ Remove-AzSqlVirtualCluster [-InputObject] <AzureSqlVirtualClusterModel> [-AsJob]
 
 ### RemoveVirtualClusterFromAzureResourceId
 ```
-Remove-AzSqlVirtualCluster [-ResourceId] <String> [-AsJob] [-DefaultProfile <IAzureContextContainer>] [-WhatIf]
+Remove-AzSqlVirtualCluster -ResourceId <String> [-AsJob] [-DefaultProfile <IAzureContextContainer>] [-WhatIf]
  [-Confirm] [<CommonParameters>]
 ```
 
@@ -128,7 +128,7 @@ Parameter Sets: RemoveVirtualClusterFromAzureResourceId
 Aliases:
 
 Required: True
-Position: 0
+Position: Named
 Default value: None
 Accept pipeline input: True (ByPropertyName)
 Accept wildcard characters: False