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.

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

[Error Handling/Codes][State Store] Add ETAG MISMATCH Error Handling #6472

Closed
Tracked by #6567
robertojrojas opened this issue Jun 8, 2023 · 4 comments
Closed
Tracked by #6567

Comments

@robertojrojas
Copy link
Contributor

robertojrojas commented Jun 8, 2023

In what area(s)?

/area runtime

Describe the feature

Add ETAG MISMATCH Error Handling following the proposal Error Handling/Codes

Release Note

RELEASE NOTE:

@dapr-bot
Copy link
Collaborator

This issue has been automatically marked as stale because it has not had activity in the last 60 days. It will be closed in the next 7 days unless it is tagged (pinned, good first issue, help wanted or triaged/resolved) or other activity occurs. Thank you for your contributions.

@dapr-bot dapr-bot added the stale Issues and PRs without response label Sep 26, 2023
@mukundansundar mukundansundar added pinned and removed stale Issues and PRs without response labels Sep 26, 2023
@cicoyle
Copy link
Contributor

cicoyle commented Jan 4, 2024

I think this can be closed due to this error codes issue already existing, with the corresponding state store error codes PR here.

@cicoyle cicoyle closed this as completed Jan 4, 2024
@elena-kolevska
Copy link
Contributor

Etag mismatch is not covered by that PR actually. Once the PR gets merged, we can keep on covering more errors, including this one.

@cicoyle
Copy link
Contributor

cicoyle commented Jan 5, 2024

I still dont think we should have an issue per error. We will open an issue per API once one of our PRs gets merged, and we can open one to finish the rest of the State API errors 👍🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants