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

Initialize reason and SQLState in PostgresqlServerErrorException. #4

Closed
wants to merge 2 commits into from

Conversation

mp911de
Copy link
Collaborator

@mp911de mp911de commented Jun 22, 2018

We now initialize reason and SQLState properties of R2dbcException
to provide details for code that uses only R2dbcException.
We're aligning the behavior with the JDBC driver that only
populates SQLState and leaves errorCode empty.

Closes gh-3.

We now initialize reason and SQLState properties of R2dbcException
to provide details for code that uses only R2dbcException.
We're aligning the behavior with the JDBC driver that only
populates SQLState and leaves errorCode empty.

Closes gh-3.
@nebhale nebhale self-assigned this Jun 22, 2018
@nebhale nebhale added this to the 1.0.0.M4 milestone Jun 22, 2018
@nebhale nebhale self-requested a review June 22, 2018 09:39
Previously, the implementation of this iterated over the collection of fields
three times.  This change iterates a single time to convert to a Map, which
then is used for efficient lookup.
Copy link
Contributor

@nebhale nebhale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change iterates over the list of fields three times. How about this?

@nebhale nebhale added the type: enhancement A general enhancement label Jun 22, 2018
@mp911de
Copy link
Collaborator Author

mp911de commented Jun 22, 2018

Looks better.

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

Successfully merging this pull request may close these issues.

Initialize reason and sqlState in PostgresqlServerErrorException
2 participants