-
Notifications
You must be signed in to change notification settings - Fork 64
Add missing Cypher 25 updates (GQL extended identifiers + subqeury expression change) #1313
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
Conversation
| } AS x | ||
| ---- | ||
|
|
||
| a| The xref:subqueries/collect.adoc[`COLLECT`], xref:subqueries/count.adoc[`COUNT`], and xref:subqueries/existential.adoc[`EXISTS`] subquery expressions no longer use imported variables as grouping variables in aggregations. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shall we say this is a change in behaviour and what happens?
| The Unicode character `\u0085` has been removed for unescaped identifiers and is now considered a whitespace character. | ||
| To continue using it, escape the identifier by adding backticks around it. | ||
| This applies to all unescaped identifiers in Cypher, such as label expressions, properties, variable names, or parameters. | ||
| In the given example, the quoted identifier would be \`my�identifier`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we fix the failed ? thingy here?
gem-neo4j
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great! just one comment :)
| [source, cypher, role="noheader"] | ||
| ---- | ||
| RETURN 1 as my$Identifier | ||
| RETURN 1 as my\u0024Identifier |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the dollar sign can stay to make it obvious what it is, I wanted the 0085 one changed because it didn't render to anything :)
modules/ROOT/pages/deprecations-additions-removals-compatibility.adoc
Outdated
Show resolved
Hide resolved
|
Thanks for the documentation updates. The preview documentation has now been torn down - reopening this PR will republish it. |
…pression change) (#1313)
No description provided.