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

Loader: Include the SQLState when logging a SQLException #941

Closed
istreeter opened this issue Jun 21, 2022 · 0 comments
Closed

Loader: Include the SQLState when logging a SQLException #941

istreeter opened this issue Jun 21, 2022 · 0 comments
Milestone

Comments

@istreeter
Copy link
Contributor

Currently, when a load fails we log the main error message from the exception. I have seen messages in the logs like:

HikariPool-1 - Connection is not available, request timed out after 30000ms.

....which comes from this line in Hikari. This is an example of a java SqlException which contains other helpful information, like the SqlState which is a dictionary of terms describing the exact reason for the failure. The logs will be much more helpful if we also log this SqlState as part of our log message.

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

No branches or pull requests

1 participant