Skip to content

feat(gql-error): add GQLSTATUS finders - #1671

Merged
injectives merged 5 commits into
neo4j:6.xfrom
injectives:feature/gqlstatus
Jul 9, 2025
Merged

injectives merged 5 commits into
neo4j:6.xfrom
injectives:feature/gqlstatus

Conversation

@injectives

Copy link
Copy Markdown
Contributor

In some situations it might be needed to check if GQL error or any error in its GQL error chain has a specific GQLSTATUS.

This update adds 2 new methods to Neo4jException to make it easier:

  • boolean containsGqlStatus(String gqlStatus)
  • Optional<Neo4jException> findByGqlStatus(String gqlStatus)

@MaxAake MaxAake left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📝

Comment thread driver/src/test/java/org/neo4j/driver/exceptions/Neo4jExceptionTest.java Outdated
Comment thread driver/src/test/java/org/neo4j/driver/exceptions/Neo4jExceptionTest.java Outdated
In some situations it might be needed to check if GQL error or any error in its GQL error chain has a specific GQLSTATUS.

This update adds 2 new methods to `Neo4jException` to make it easier:
- `boolean containsGqlStatus(String gqlStatus)`
- `Optional<Neo4jException> findByGqlStatus(String gqlStatus)`
@injectives
injectives force-pushed the feature/gqlstatus branch from e98673b to ceb94ca Compare July 8, 2025 11:37
@injectives
injectives requested a review from MaxAake July 8, 2025 11:37

@MaxAake MaxAake left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM👍🏻

@Lojjs Lojjs left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thank you so much, looks great to me!

Comment thread driver/src/main/java/org/neo4j/driver/exceptions/Neo4jException.java Outdated
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

Successfully merging this pull request may close these issues.

4 participants