From e33ebf891469b27ed38f0de2d8cb37ae5d12f722 Mon Sep 17 00:00:00 2001 From: Joo Wan Ro Date: Fri, 25 Aug 2017 22:04:06 -0700 Subject: [PATCH] [BUGFIX] - Reverting readOnly for two fields (#1595) * Refactoring changes in order to match with Ben's refactoring changes * There is no functional change in this commit * This change is purely for refactoring related to Ben's refactoring changes (which has already been accepted): https://github.com/Azure/azure-rest-api-specs/pull/1429 * Reordering fields in the example files to match with Ben's refactoring changes * Other minor changes or moving code around * [BUGFIX] - Reverting readOnly for two fields * In this already-merged PR (https://github.com/Azure/azure-rest-api-specs/pull/1429), Ben made some object-refactoring changes - one of them being to make all the fields in USqlJobProperties read-only as a good design decision. However, by making the fields statistics and debugData read-only, this caused some conflicts in the Powershell code. We need to pan out the design more carefully for these two fields with respect to their associated APIs ("/Jobs/{jobIdentity}/GetDebugDataPath" for the debugData field and "/Jobs/{jobIdentity}/GetStatistics" for the statistics field) before making them read-only. If I do not make the change proposed here, I would have to create two new cmdlets: one for getting the JobDataPath object and another for getting the JobStatistics object, which we do not want to do until we have a better understanding of their designs as a whole. Hence, the reverting change proposed here. --- .../data-plane/Microsoft.DataLakeAnalytics/2016-11-01/job.json | 2 -- 1 file changed, 2 deletions(-) diff --git a/specification/datalake-analytics/data-plane/Microsoft.DataLakeAnalytics/2016-11-01/job.json b/specification/datalake-analytics/data-plane/Microsoft.DataLakeAnalytics/2016-11-01/job.json index eba983c46077..aa175b844b11 100644 --- a/specification/datalake-analytics/data-plane/Microsoft.DataLakeAnalytics/2016-11-01/job.json +++ b/specification/datalake-analytics/data-plane/Microsoft.DataLakeAnalytics/2016-11-01/job.json @@ -747,12 +747,10 @@ }, "statistics": { "$ref": "#/definitions/JobStatistics", - "readOnly": true, "description": "the job specific statistics." }, "debugData": { "$ref": "#/definitions/JobDataPath", - "readOnly": true, "description": "the job specific debug data locations." }, "diagnostics": {