Skip to content

Conversation

injectives
Copy link
Contributor

@injectives injectives commented Jun 1, 2024

This update adds support for GQL-status objects defined in the GQL standard.

@injectives injectives marked this pull request as draft June 1, 2024 12:28
@injectives injectives force-pushed the feature/gql branch 5 times, most recently from 5498dd5 to 3137198 Compare June 1, 2024 14:39
@injectives injectives changed the title Add support for GQL Status Object Add support for GQL-status objects Jun 11, 2024
@injectives injectives force-pushed the feature/gql branch 17 times, most recently from b336322 to 667231b Compare June 12, 2024 15:41
@injectives injectives marked this pull request as ready for review June 12, 2024 15:41
This update adds support for GQL-status objects defined in the GQL standard.
Copy link
Contributor

@bigmontz bigmontz left a comment

Choose a reason for hiding this comment

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

We should not deprecate the current apis yet and the new apis should be released as preview.

@injectives injectives requested a review from bigmontz June 13, 2024 13:36
@injectives injectives requested a review from bigmontz June 13, 2024 14:01
Copy link
Contributor

@bigmontz bigmontz left a comment

Choose a reason for hiding this comment

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

🔔

Copy link
Contributor

@meistermeier meistermeier left a comment

Choose a reason for hiding this comment

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

Completely ignored the bits around the communication layer because I have not much knowledge here.
Some remarks on dependencies and inheritance.

@injectives injectives merged commit e5d1bfe into neo4j:5.0 Jun 20, 2024
@injectives injectives deleted the feature/gql branch June 20, 2024 09:54
injectives added a commit to injectives/neo4j-java-driver that referenced this pull request Jul 28, 2024
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.

3 participants