-
Notifications
You must be signed in to change notification settings - Fork 5.4k
Update HDInsight Linked Service Swagger definition with new payload f… #36664
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
…ields: {clusterAuthType, credential}
Next Steps to MergeNext steps that must be taken to merge this PR:
Comment generated by summarize-checks workflow run. |
API Change CheckAPIView identified API level changes in this PR and created the following API reviews
|
@nakul16168 The Breaking Change validation for this PR flagged OAD violations during same-version comparison of LinkedService.json. Based on the report, these appear to be additions of properties. So, please confirm / justify that. Given this, VersioningReviewRequired label seems appropriate. In that case:
@mentat9 for awareness. |
…ields: {clusterAuthType, credential}
Choose a PR Template
Switch to "Preview" on this description then select one of the choices below.
Click here to open a PR for a Data Plane API.
Click here to open a PR for a Control Plane (ARM) API.
Click here to open a PR for only SDK configuration.