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

Improve error messages #7533

Closed
3 tasks done
deepthi opened this issue Feb 22, 2021 · 3 comments
Closed
3 tasks done

Improve error messages #7533

deepthi opened this issue Feb 22, 2021 · 3 comments

Comments

@deepthi
Copy link
Member

deepthi commented Feb 22, 2021

Feature Description

Right now we produce errors like "no valid tablet", which are accurate but not very helpful. Sometimes the error messages also contain references to internal functions like execInsertUnsharded.

  • Remove vitess-internal texts from error messages (e.g. function names)
  • Every error must have some error code, and should closely match the type of error defined by MySQL. If possible also return root cause.
  • Document all error messages and root causes Error documentation website#750

Use Case(s)

VTAdmin
Usability

@derekperkins
Copy link
Member

This would be very useful to support on the Go driver side. We currently have a ton of text parsing to recreate the original proto error struct.

@GuptaManan100
Copy link
Member

Is there something pending on this issue? We are now generating documentation for the website automatically.

@harshit-gangal
Copy link
Member

The scoped work is completed.

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

No branches or pull requests

6 participants