Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Tables] Merge operation fails with a Cosmos endpoint #13844

Closed
seankane-msft opened this issue Sep 17, 2020 · 1 comment · Fixed by #17888
Closed

[Tables] Merge operation fails with a Cosmos endpoint #13844

seankane-msft opened this issue Sep 17, 2020 · 1 comment · Fixed by #17888
Assignees
@seankane-msft
Labels
Client This issue points to a problem in the data-plane of the library. Tables
Milestone
Backlog

Comments

@seankane-msft
Copy link
Member

The Merge operation (which is implemented as Patch) always fails with a 405 MethodNotAllowed response against Cosmos table endpoints. Inspecting the traffic sent by the track 1 client reveals that they send Merge as Post with an additional header: X-HTTP-Method: MERGE . Adding this header and changing the method to Post results in a successful merge in my testing.

Ideally the Cosmos Table endpoint would accept the same http request shape as Storage.

#cosmosKnownIssue

.NET Issue
@seankane-msft seankane-msft added the Mgmt This issue is related to a management-plane library. label Sep 17, 2020
@seankane-msft seankane-msft added this to the Backlog milestone Sep 17, 2020
@seankane-msft seankane-msft self-assigned this Sep 17, 2020
@ghost
Copy link

ghost commented Sep 17, 2020

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @wmengmsft, @MehaKaushik, @shurd, @anfeldma-ms

@seankane-msft seankane-msft added Client This issue points to a problem in the data-plane of the library. Tables and removed Mgmt This issue is related to a management-plane library. labels Sep 21, 2020
@lmazuel lmazuel mentioned this issue Oct 2, 2020
Merged
@annatisch annatisch mentioned this issue Apr 8, 2021
Merged
@ghost ghost closed this as completed in #17888 Apr 8, 2021
ghost pushed a commit that referenced this issue Apr 8, 2021
@seankane-msft
[Tables] Cosmos workaround for Merge/Update (#17888)
cb464b2 
Closes #13844
mccoyp pushed a commit to mccoyp/azure-sdk-for-python that referenced this issue Apr 9, 2021
@seankane-msft @mccoyp
[Tables] Cosmos workaround for Merge/Update (Azure#17888)
c88aa1d 
Closes Azure#13844
mccoyp pushed a commit to mccoyp/azure-sdk-for-python that referenced this issue Apr 13, 2021
@seankane-msft @mccoyp
[Tables] Cosmos workaround for Merge/Update (Azure#17888)
dd3a03f 
Closes Azure#13844
benbp pushed a commit to benbp/azure-sdk-for-python that referenced this issue Apr 19, 2021
@seankane-msft @benbp
[Tables] Cosmos workaround for Merge/Update (Azure#17888)
f9cc68c 
Closes Azure#13844
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@seankane-msft seankane-msft

Labels
Client This issue points to a problem in the data-plane of the library. Tables
Projects
None yet
Milestone
Backlog
Development

Successfully merging a pull request may close this issue.

[Tables] Cosmos workaround for Merge/Update seankane-msft/azure-sdk-for-python
1 participant
@seankane-msft