Address PR feedback.

Author: begoldsm

src/ResourceManager/DataLakeAnalytics/ChangeLog.md

@@ -18,6 +18,11 @@
         - Additional information about change #1
 -->
 ## Current Release
+* Update Get-AdlJob to support Top parameter
+* Update Get-AdlJob to return the list of jobs in order by most recently submitted
+* Updated help for all cmdlets to include output as well as more descriptions of parameters and the inclusion of aliases.
+* Update New-AdlAnalyticsAccount and Set-AdlAnalyticsAccount to support commitment tier options for the service.
+* Added OutputType mismatch warnings to all cmdlets with incorrect OutputType attributes. These will be fixed in a future breaking change release.
 
 ## Version 2.4.0
 * Removal of unsupported parameters in Add and Set-AzureRMDataLakeAnalyticsDataSource (default for data lake store)

src/ResourceManager/DataLakeAnalytics/Commands.DataLakeAnalytics/Commands/GetAzureRmDataLakeAnalyticsJob.cs

@@ -90,7 +90,6 @@ public class GetAzureDataLakeAnalyticsJob : DataLakeAnalyticsCmdletBase
         [Parameter(ParameterSetName = BaseParameterSetName, ValueFromPipelineByPropertyName = true,
             Mandatory = false, HelpMessage = "An optional value which indicates the number of jobs to return. Default value is 500")]
         [ValidateNotNullOrEmpty]
-
         public int? Top { get; set; }
 
         public override void ExecuteCmdlet()

src/ResourceManager/DataLakeAnalytics/Commands.DataLakeAnalytics/Commands/SetAzureRmDataLakeAnalyticsCatalogSecret.cs

@@ -73,8 +73,10 @@ public override void ExecuteCmdlet()
 
             var toUse = Uri ?? new Uri(string.Format("https://{0}:{1}", DatabaseHost, Port));
 
-            DataLakeAnalyticsClient.UpdateSecret(Account, DatabaseName, Secret.UserName,
-                Secret.GetNetworkCredential().Password, toUse.AbsoluteUri);
+            // TODO: Remove the WriteObject during next breaking change release, since this object
+            // is always null.
+            WriteObject(DataLakeAnalyticsClient.UpdateSecret(Account, DatabaseName, Secret.UserName,
+                Secret.GetNetworkCredential().Password, toUse.AbsoluteUri));
         }
     }
 }

src/ResourceManager/DataLakeAnalytics/Commands.DataLakeAnalytics/Models/DataLakeAnalyticsClient.cs

@@ -411,7 +411,7 @@ public void CreateSecret(string accountName, string databaseName,
                 });
         }
 
-        public void UpdateSecret(string accountName, string databaseName,
+        public USqlSecret UpdateSecret(string accountName, string databaseName,
             string secretName, string password, string hostUri)
         {
             _catalogClient.Catalog.UpdateSecret(accountName, databaseName, secretName,
@@ -420,6 +420,9 @@ public void UpdateSecret(string accountName, string databaseName,
                     Password = password,
                     Uri = hostUri
                 });
+
+            // TODO: Remove this during the next breaking change release.
+            return null;
         }
 
         public void DeleteSecret(string accountName, string databaseName, string secretName)

src/ResourceManager/DataLakeStore/ChangeLog.md

@@ -18,6 +18,14 @@
         - Additional information about change #1
 -->
 ## Current Release
+* Updated help for all cmdlets to include output as well as more descriptions of parameters and the inclusion of aliases.
+* Update New-AdlStore and Set-AdlStore to support commitment tier options for the service.
+* Added OutputType mismatch warnings to all cmdlets with incorrect OutputType attributes. These will be fixed in a future breaking change release.
+* Add Diagnostic logging support to Import-AdlStoreItem and Export-AdlStoreItem. This can be enabled through the following parameters:
+  * -Debug, enables full diagnostic logging as well as debug logging to the PowerShell console. Most verbose options
+  * -DiagnosticLogLevel, allows finer control of the output than debug. If used with debug, this is ignored and debug logging is used.
+  * -DiagnosticLogPath, optionally specify the file to write diagnostic logs to. By default it is written to a file under %LOCALAPPDATA%\AdlDataTransfer
+* Added support to New-AdlStore to explicitly opt-out of account encryption. To do so, create the account with the -DisableEncryption flag.
 
 ## Version 3.2.0
 * Introduction of deprecation warning for nested properties for all ARM resources. Nested properties will be removed in a future release and all properties will be moved one level up.