Skip to content

fix: Enhance the expressiveness of VertexAccessor and EdgeAccessor #215

Merged
liulx20 merged 8 commits intoalibaba:mainfrom
liulx20:vertex-var
Apr 14, 2026
Merged

fix: Enhance the expressiveness of VertexAccessor and EdgeAccessor #215
liulx20 merged 8 commits intoalibaba:mainfrom
liulx20:vertex-var

Conversation

@liulx20
Copy link
Copy Markdown
Collaborator

@liulx20 liulx20 commented Apr 13, 2026

This pull request adds support for identity accessors for both vertices and edges, enabling retrieval of the entire vertex or edge object when no specific property or ID is requested. It also updates the variable expression parsing logic to use these identity accessors instead of aborting on unsupported property accesses. Additionally, a new test is introduced to ensure that checking for node variable nullability does not cause a runtime abort.
Fixes

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@liulx20 liulx20 requested a review from shirly121 April 13, 2026 05:02
@liulx20 liulx20 changed the title fix: Enhance the expressiveness of VertexAccessor and EdgeAccessor to supp… fix: Enhance the expressiveness of VertexAccessor and EdgeAccessor Apr 13, 2026
@liulx20 liulx20 merged commit 189a6e8 into alibaba:main Apr 14, 2026
16 checks passed
@liulx20 liulx20 deleted the vertex-var branch April 14, 2026 07:04
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.

[BUG] Node null-check predicate aborts with vertex variable missing property

2 participants