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

Path patterns support #31

Draft
wants to merge 22 commits into
base: main
Choose a base branch
from

Conversation

gsvgit
Copy link

@gsvgit gsvgit commented Jul 2, 2020

The syntax for path patterns as described in this proposal (with this fix) is supported. This extension allows one to express RPQs and CFPQs (regular and context-free path queries) in Cypher.

The previous request which contains some discussion, was closed during branches renaming.

@cleishm
Copy link
Owner

cleishm commented Jul 27, 2020

Sorry for the delay on this one. I've been busy and not paying a lot of attention to these projects. I'll try to look at it again soon!

@gsvgit
Copy link
Author

gsvgit commented Jul 28, 2020

Great! Waiting for your feedback.

@cleishm cleishm marked this pull request as draft November 11, 2021 04:35
@cleishm
Copy link
Owner

cleishm commented Nov 11, 2021

After a sabbatical from coding, I'm getting back into things by looking over some outstanding code. This is one of those.

I've done some updates to get this code (mostly) working, as well as proposing some changes to the AST structure, which are in a commit here: 9c35a7e. The branch https://github.com/cleishm/libcypher-parser/tree/path-patterns-support is this PR merged onto the current main branch.

If you're still interested in the PR, I'd be happy to discuss the proposed AST format and give thoughts and some effort toward implementing.

@Timur-Zinnatulin
Copy link

Hello, Chris.

Thank you very much for your feedback! We are looking forward to working with you on this extension and would be happy to hear your thoughts on it.

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