Azure / azure-rest-api-specs

The source for REST API specifications for Microsoft Azure.
MIT License
2.69k stars 5.12k forks source link

no available type checker for linked service properties #25741

Open ChenxiJiang333 opened 1 year ago

ChenxiJiang333 commented 1 year ago

originally opened in https://github.com/Azure/azure-sdk-for-python/issues/31975

swagger definition: https://github.com/Azure/azure-rest-api-specs/blob/main/specification/datafactory/resource-manager/Microsoft.DataFactory/stable/2018-06-01/entityTypes/LinkedService.json#L6707-L6709

Type checker defined here didn't work as expected. Properties can still be passed with value in any types other than string. image

For example, as the log snippet shows, newClusterNumOfWorker can be passed with an integer though expected a string.

Request body:
{"properties": {"type": "AzureDatabricks", "typeProperties": {"domain": "https://adb-xxxxxxxxxxxxxx.x.azuredatabricks.net", "newClusterVersion": "13.3.x-scala2.12", "newClusterNumOfWorker": 2, "newClusterNodeType": "Standard_DS3_v2"}}}

Expected behavior Type checker for type properties of linked service should be updated to be able to restrict the properties' value in string format.

microsoft-github-policy-service[bot] commented 1 year ago

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @Jingshu923, @zhangyd2015, @Frey-Wang.