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
Add SQLState to MySQLError #1321
Add SQLState to MySQLError #1321
Conversation
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.
it needs some little memory allocations, but not so frequently.
#1322 is acceptable for me.
@jmhodges @arnehormann @methane How do you think about it?
4ab8a7a
to
14893f1
Compare
14893f1
to
c0031c2
Compare
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.
👍🏼
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.
Please remove IsWithSQLState.
to allow library users to distinguish user-defined from client / server errors
c0031c2
to
0e8599a
Compare
Thank you @shogo82148 and @methane for your patience and thorough review. |
Description
Allow library users to distinguish user-defined from client / server errors.
Checklist