Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
[Hub Generated] Review request for Microsoft.DocumentDB to add version preview/2021-10-15-preview #16211
[Hub Generated] Review request for Microsoft.DocumentDB to add version preview/2021-10-15-preview #16211
Changes from 24 commits
b022773
3207664
1db5652
60844c7
8392017
ad69995
f2491e1
880e31e
53e213e
c6bf669
23e0c72
287e49f
0e69bc6
90934e4
78ccf43
e112c4b
9d6dc8a
6c7d61a
5c52c9c
76e8fca
97f14ed
ba8da86
5f8274e
5c95253
b402c75
25b9827
56a87a1
fd3af21
f9ec011
192f8e9
b3054c1
06a4af2
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Large diffs are not rendered by default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this is not a long running operation, you can just return 200 instead of 201.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will require a fix on management service. I have pushed the fix in the pipeline, but deployment might take some time. As an exception can we merge this in the preview version and later fix it in GA.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Success response is 200. 201 is optional to indicate first time creation.