Skip to content

Create correct union statements. #2178

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.

Sign up for GitHub

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

Jump to bottom
Closed
meistermeier opened this issue Mar 12, 2021 · 0 comments
Closed

Create correct union statements. #2178

meistermeier opened this issue Mar 12, 2021 · 0 comments
Assignees
@meistermeier
Labels
type: bug A general bug
Milestone
6.0.6 (2020.0.6)

Comments

@meistermeier
Copy link
Collaborator

This is part of GH-2177

Apart from the version problem, I find the ...UNION MATCH (n) WHERE (n.name = $id AND n.version = $version)... quite strange because if my DB has other different nodes (Posts, Groups...) with name property they will take into account in the query. Wouldn't it be better something like ...UNION MATCH (n:User) WHERE...?

This is due to a bug in the CypherDSL but a workaround in SDN is possible.
@meistermeier meistermeier self-assigned this Mar 12, 2021
meistermeier added a commit that referenced this issue Mar 12, 2021
@meistermeier
GH-2178 - Create correct union statements.
6ad4628 
This closes GH-2178
meistermeier added a commit that referenced this issue Mar 16, 2021
@meistermeier
GH-2178 - Remove union creation fallback.
b3adab6
meistermeier added a commit that referenced this issue Mar 16, 2021
@meistermeier
GH-2178 - Remove union creation fallback.
fd20a81
@meistermeier meistermeier added this to the 6.0.6 (2020.0.6) milestone Mar 16, 2021
@michael-simons michael-simons added type: bug A general bug and removed bug labels Apr 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@meistermeier meistermeier

Labels
type: bug A general bug
Projects
None yet
Milestone
6.0.6 (2020.0.6)
Development

No branches or pull requests
2 participants
@meistermeier @michael-simons