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

Fix issue 1393 - previous clause variables not seen with EXISTS (#1426) #1430

Merged
merged 1 commit into from
Nov 29, 2023

Conversation

jrgemignani
Copy link
Contributor

Fixed issue 1393 - Unexpected error when matching nodes that are connected to nodes resulted from a previous MATCH clause.

Basically, variables from the previous clauses where not seen when using the EXISTS clause for path pattern matches. This only affected EXISTS and only for the EXISTS (anonymous_path) grammar component.

The original issue was only with vertices, but this also affected edges. Both cases are now fixed.

Fixed 1 regression test that was incorrect.
Added regression tests.

Resolved -

Conflicts:
src/backend/parser/cypher_clause.c

@github-actions github-actions bot added the PG11 PostgreSQL11 label Nov 28, 2023
…he#1426)

Fixed issue 1393 - Unexpected error when matching nodes that are
connected to nodes resulted from a previous MATCH clause.

Basically, variables from the previous clauses where not seen when
using the EXISTS clause for path pattern matches. This only affected
EXISTS and only for the EXISTS (anonymous_path) grammar component.

The original issue was only with vertices, but this also affected
edges. Both cases are now fixed.

Fixed 1 regression test that was incorrect.
Added regression tests.

Resolved -

Conflicts:
	src/backend/parser/cypher_clause.c
Copy link
Member

@dehowef dehowef left a comment

Choose a reason for hiding this comment

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

Looks good to me

@dehowef dehowef merged commit 88bd5a8 into apache:PG11 Nov 29, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PG11 PostgreSQL11
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants