-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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
Feature Request : A Status Code to indicate Success or Failure (instead of just errorCode in event of failure) #83
Comments
@conikeec Http status code is different. Is it enough for your usage? |
I am currently using HTTP codes the terminate flow. |
@conikeec First, I feel return response status code is a little bit duplicate with http status code. |
Got it .. Agree with you ... |
@conikeec If you have any questions, just open an issue. PR and issues are always welcomed. |
snapshot: move pb to snappb; remove clusterId
ETCD-199: bump etcd v3.4.16
In event of a network failure, an insert into etcd might fail, but an external process might query of an non-existent key for variety of reasons.
Non-existent keys send a distinct failed message:
{"errorCode":100,"message":"Key Not Found","cause":"/D0A1A866-6591-40CF-99EE-15AE1BF3A7F1"}
Successful value fetch from key does not hold a statusCode at this point
[{"action":"GET","key":"/51b8cc7c-c7d8-4d6e-86ee-1a381db2f068/status","value":"COMPLETED","expiration":"2013-08-11T06:04:23.647167583-07:00","ttl":29849,"index":97}]
Is it possible to add an upper level 'statusCode' to indicate success or failure instead of just passing errorCode on event of a failure?
Client(s) can then use the statusCode to make deterministic logic flow.
The text was updated successfully, but these errors were encountered: