Merged ADMS changes for Nov. SDK release#3824
Conversation
markcowl
left a comment
There was a problem hiding this comment.
You will need to update the version, also, where are the covering tests for the API changes?
|
|
||
| ### Feature Additions | ||
| * Cancel pipeline run api. | ||
| * Add AzureMySql linked service. |
There was a problem hiding this comment.
You will need to update the version for these changes in your csproj file (0.3.0-preview or 0.2.1-preview).
There was a problem hiding this comment.
We'll be bumping the version in a separate change soon (to 0.2.1-preview), thanks for the reminder
| } | ||
| } | ||
| "; | ||
| [JsonSample] |
There was a problem hiding this comment.
Thanks for the samples, but I don't see any actual test updates
There was a problem hiding this comment.
these run automatically as part of LinkedService_SerializationTest, Dataset_SerializationTest and Activity_SerializationTest
There was a problem hiding this comment.
see the ClassData attribute on each
Description
This is a merged PR for three SDK changes from ADMS side.
Related features and corresponding swagger changes, SDK changes and internal items are:
AzureMySql connector:
[Swagger PR] Azure/azure-rest-api-specs#1911
[Internal item] 97894
[Previous SDK PR] #3813
Dynamics, Salesforce and JSONFormat:
[Swagger PR] Azure/azure-rest-api-specs#1850
Azure/azure-rest-api-specs#1903
[Internal item] 97878
[Previous SDK PR] #3811
Support providing Salesforce passwords and security tokens as SecureStrings or as secrets in Azure Key Vault:
[Swagger PR] Azure/azure-rest-api-specs#1917
[Internal item] 97519
[Previous SDK PR] #3816
This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines
Testing Guidelines
SDK Generation Guidelines
*.csprojandAssemblyInfo.csfiles have been updated with the new version of the SDK.