Skip to content

[Azure Cognitive Search] Creating new SDK version to match new management-plane API Version 2020-03-13#12376

Closed
tjacobhi wants to merge 0 commit intoAzure:masterfrom
tjacobhi:master
Closed

[Azure Cognitive Search] Creating new SDK version to match new management-plane API Version 2020-03-13#12376
tjacobhi wants to merge 0 commit intoAzure:masterfrom
tjacobhi:master

Conversation

@tjacobhi
Copy link
Copy Markdown
Contributor

SDK version to point to swagger changes from this PR - Updated default tag to latest stable version. #9085.

Due to data-plane tests being dependent on the generated management-plane code, all session records were re-recorded. This PR contains no changes to generated data-plane SDK code. There is planned work to separate the dependency between data-plane tests and management-plane generated code.

@isra-fel isra-fel self-assigned this Jun 1, 2020
@isra-fel isra-fel added Mgmt This issue is related to a management package. needs-review labels Jun 1, 2020
Copy link
Copy Markdown
Member

@isra-fel isra-fel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @tjacobhi mangement plane code looks good to me. However there are some data plane test failing and a conflicting file. Please reslove them. Thanks

Copy link
Copy Markdown
Member

@jsquire jsquire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please consider rebasing on master. I should not be seeing any changes to the Event Hubs data plane library included as part of this work.

@tjacobhi
Copy link
Copy Markdown
Contributor Author

tjacobhi commented Jun 8, 2020

Sorry, I rebased when I meant to merge in the recent changes to master. I will fix this quickly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Mgmt This issue is related to a management package.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants