Skip to content

Commit 0c130e1

Browse files
nemanja88nemanja88
committed
Merge branch 'rickhall-hdicmdlet_persistconfig' of https://github.com/rickmsft/azure-powershell into vmsizes
2 parents 81136ba + 7ac7aac commit 0c130e1

File tree

3 files changed

+47
-27
lines changed

3 files changed

+47
-27
lines changed

‎src/ServiceManagement/HDInsight/Commands.HDInsight/Cmdlet/AddAzureHDInsightMetastoreCmdlet.cs

Lines changed: 17 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -58,18 +58,27 @@ public AzureHDInsightConfig Config
5858
{
5959
throw new ArgumentNullException("value", "The value for the configuration can not be null.");
6060
}
61+
6162
this.command.Config.ClusterSizeInNodes = value.ClusterSizeInNodes;
6263
this.command.Config.DefaultStorageAccount = value.DefaultStorageAccount;
6364
this.command.Config.AdditionalStorageAccounts.AddRange(value.AdditionalStorageAccounts);
6465
this.command.Config.ConfigActions.AddRange(value.ConfigActions);
65-
if (value.HiveMetastore.IsNotNull())
66-
{
67-
this.command.Config.HiveMetastore = value.HiveMetastore;
68-
}
69-
if (value.OozieMetastore.IsNotNull())
70-
{
71-
this.command.Config.OozieMetastore = value.OozieMetastore;
72-
}
66+
this.command.Config.HiveMetastore = value.HiveMetastore ?? this.command.Config.HiveMetastore;
67+
this.command.Config.OozieMetastore = value.OozieMetastore ?? this.command.Config.OozieMetastore;
68+
this.command.Config.CoreConfiguration.AddRange(value.CoreConfiguration);
69+
this.command.Config.YarnConfiguration.AddRange(value.YarnConfiguration);
70+
this.command.Config.HdfsConfiguration.AddRange(value.HdfsConfiguration);
71+
this.command.Config.MapReduceConfiguration.ConfigurationCollection.AddRange(value.MapReduceConfiguration.ConfigurationCollection);
72+
this.command.Config.MapReduceConfiguration.CapacitySchedulerConfigurationCollection.AddRange(
73+
value.MapReduceConfiguration.CapacitySchedulerConfigurationCollection);
74+
this.command.Config.HiveConfiguration.ConfigurationCollection.AddRange(value.HiveConfiguration.ConfigurationCollection);
75+
this.command.Config.OozieConfiguration.ConfigurationCollection.AddRange(value.OozieConfiguration.ConfigurationCollection);
76+
this.command.Config.HeadNodeVMSize = value.HeadNodeVMSize;
77+
this.command.Config.ClusterType = value.ClusterType;
78+
this.command.Config.VirtualNetworkId = value.VirtualNetworkId;
79+
this.command.Config.SubnetName = value.SubnetName;
80+
this.command.Config.StormConfiguration.AddRange(value.StormConfiguration);
81+
this.command.Config.HBaseConfiguration.ConfigurationCollection.AddRange(value.HBaseConfiguration.ConfigurationCollection);
7382
}
7483
}
7584

‎src/ServiceManagement/HDInsight/Commands.HDInsight/Cmdlet/AddAzureHDInsightStorageCmdlet.cs

Lines changed: 17 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,12 +58,27 @@ public AzureHDInsightConfig Config
5858
{
5959
throw new ArgumentNullException("value", "The value for the configuration can not be null.");
6060
}
61+
6162
this.command.Config.ClusterSizeInNodes = value.ClusterSizeInNodes;
6263
this.command.Config.DefaultStorageAccount = value.DefaultStorageAccount;
63-
this.command.Config.HiveMetastore = value.HiveMetastore ?? this.command.Config.HiveMetastore;
64-
this.command.Config.OozieMetastore = value.OozieMetastore ?? this.command.Config.OozieMetastore;
6564
this.command.Config.AdditionalStorageAccounts.AddRange(value.AdditionalStorageAccounts);
6665
this.command.Config.ConfigActions.AddRange(value.ConfigActions);
66+
this.command.Config.HiveMetastore = value.HiveMetastore ?? this.command.Config.HiveMetastore;
67+
this.command.Config.OozieMetastore = value.OozieMetastore ?? this.command.Config.OozieMetastore;
68+
this.command.Config.CoreConfiguration.AddRange(value.CoreConfiguration);
69+
this.command.Config.YarnConfiguration.AddRange(value.YarnConfiguration);
70+
this.command.Config.HdfsConfiguration.AddRange(value.HdfsConfiguration);
71+
this.command.Config.MapReduceConfiguration.ConfigurationCollection.AddRange(value.MapReduceConfiguration.ConfigurationCollection);
72+
this.command.Config.MapReduceConfiguration.CapacitySchedulerConfigurationCollection.AddRange(
73+
value.MapReduceConfiguration.CapacitySchedulerConfigurationCollection);
74+
this.command.Config.HiveConfiguration.ConfigurationCollection.AddRange(value.HiveConfiguration.ConfigurationCollection);
75+
this.command.Config.OozieConfiguration.ConfigurationCollection.AddRange(value.OozieConfiguration.ConfigurationCollection);
76+
this.command.Config.HeadNodeVMSize = value.HeadNodeVMSize;
77+
this.command.Config.ClusterType = value.ClusterType;
78+
this.command.Config.VirtualNetworkId = value.VirtualNetworkId;
79+
this.command.Config.SubnetName = value.SubnetName;
80+
this.command.Config.StormConfiguration.AddRange(value.StormConfiguration);
81+
this.command.Config.HBaseConfiguration.ConfigurationCollection.AddRange(value.HBaseConfiguration.ConfigurationCollection);
6782
}
6883
}
6984

‎src/ServiceManagement/HDInsight/Commands.HDInsight/Cmdlet/SetAzureHDInsightDefaultStorageCmdlet.cs

Lines changed: 13 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -56,30 +56,26 @@ public AzureHDInsightConfig Config
5656
throw new ArgumentNullException("value", "The value for the configuration can not be null.");
5757
}
5858

59-
if (value.CoreConfiguration != null)
60-
{
61-
this.command.Config.CoreConfiguration.AddRange(value.CoreConfiguration);
62-
}
63-
64-
if (value.YarnConfiguration != null)
65-
{
66-
this.command.Config.YarnConfiguration.AddRange(value.YarnConfiguration);
67-
}
6859
this.command.Config.ClusterSizeInNodes = value.ClusterSizeInNodes;
60+
this.command.Config.DefaultStorageAccount = value.DefaultStorageAccount;
6961
this.command.Config.AdditionalStorageAccounts.AddRange(value.AdditionalStorageAccounts);
7062
this.command.Config.ConfigActions.AddRange(value.ConfigActions);
7163
this.command.Config.HiveMetastore = value.HiveMetastore ?? this.command.Config.HiveMetastore;
7264
this.command.Config.OozieMetastore = value.OozieMetastore ?? this.command.Config.OozieMetastore;
65+
this.command.Config.CoreConfiguration.AddRange(value.CoreConfiguration);
66+
this.command.Config.YarnConfiguration.AddRange(value.YarnConfiguration);
67+
this.command.Config.HdfsConfiguration.AddRange(value.HdfsConfiguration);
68+
this.command.Config.MapReduceConfiguration.ConfigurationCollection.AddRange(value.MapReduceConfiguration.ConfigurationCollection);
69+
this.command.Config.MapReduceConfiguration.CapacitySchedulerConfigurationCollection.AddRange(
70+
value.MapReduceConfiguration.CapacitySchedulerConfigurationCollection);
71+
this.command.Config.HiveConfiguration.ConfigurationCollection.AddRange(value.HiveConfiguration.ConfigurationCollection);
72+
this.command.Config.OozieConfiguration.ConfigurationCollection.AddRange(value.OozieConfiguration.ConfigurationCollection);
7373
this.command.Config.HeadNodeVMSize = value.HeadNodeVMSize;
7474
this.command.Config.ClusterType = value.ClusterType;
75-
if (!string.IsNullOrEmpty(value.VirtualNetworkId))
76-
{
77-
this.command.Config.VirtualNetworkId = value.VirtualNetworkId;
78-
}
79-
if (!string.IsNullOrEmpty(value.SubnetName))
80-
{
81-
this.command.Config.SubnetName = value.SubnetName;
82-
}
75+
this.command.Config.VirtualNetworkId = value.VirtualNetworkId;
76+
this.command.Config.SubnetName = value.SubnetName;
77+
this.command.Config.StormConfiguration.AddRange(value.StormConfiguration);
78+
this.command.Config.HBaseConfiguration.ConfigurationCollection.AddRange(value.HBaseConfiguration.ConfigurationCollection);
8379
}
8480
}
8581

0 commit comments

Comments
 (0)