-
Notifications
You must be signed in to change notification settings - Fork 60
graph references in Cypher 25 #1295
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
base: cypher-25
Are you sure you want to change the base?
Changes from 2 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -174,20 +174,14 @@ label:functionality[] | |||||||||||||||
label:updated[] | ||||||||||||||||
[source, cypher, role="noheader"] | ||||||||||||||||
---- | ||||||||||||||||
USE graph.byName('tom`s-database') | ||||||||||||||||
|
||||||||||||||||
USE graph.propertiesByName('database.with.dot') | ||||||||||||||||
USE graph.propertiesByName('`composite.with.dot`.constituent') | ||||||||||||||||
---- | ||||||||||||||||
| xref::values-and-types/graph-references.adoc[Graph references] in arguments of the functions xref:functions/graph.adoc#functions-graph-byname[`graph.byName`] and xref:functions/graph.adoc#functions-graph-propertiesByName[`graph.propertiesByName`] in Cypher 25 are parsed as `<symbolic-name>` or `<symbolic-name>.<symbolic-name>` and now support escaping names. | ||||||||||||||||
For more information, see xref:syntax/expressions.adoc#graph-references[Cypher expressions -> Graph references]. | ||||||||||||||||
Graph name parts that contain unsupported characters for unescaped symbolic names now require backtick quoting. | ||||||||||||||||
Graph name parts with special characters may require additional escaping of those characters: | ||||||||||||||||
|
||||||||||||||||
* `++USE graph.byName('`tom``s-database`')++` | ||||||||||||||||
|
||||||||||||||||
* `++USE graph.propertiesByName('database.with.dot')++` | ||||||||||||||||
| | ||||||||||||||||
The graph name ``composite.with.dot`.constituent` is deprecated. It is replaced by ``composite.with.dot`.constituent`. | ||||||||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. isn't your replacement the exact same thing as the deprecated one? 🤔 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||||||||||||||||
|
||||||||||||||||
Note that escaping graph names within the graph functions string argument is not supported in Cypher 5. | ||||||||||||||||
In Cypher 5, there was some special logic for composite databases or constituents with dots, | ||||||||||||||||
which required quotes around those name parts in order to find the name. This deprecations apply to xref::values-and-types/graph-references.adoc[Graph references] in arguments of the functions xref:functions/graph.adoc#functions-graph-byname[`graph.byName`] and xref:functions/graph.adoc#functions-graph-propertiesByName[`graph.propertiesByName`]. In Cypher 25 the graph references can be passed in without quoting | ||||||||||||||||
as in the name column from `SHOW ALIASES FOR DATABASE`.Note that escaping graph names within the graph functions string argument is not supported in Cypher 5. | ||||||||||||||||
Comment on lines
+182
to
+184
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Usually we split the lines between sentences and not in the middle of sentences |
||||||||||||||||
|
||||||||||||||||
a| | ||||||||||||||||
label:functionality[] | ||||||||||||||||
|
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.
since this is talking about deprecations/removals it should probably not be under the
updated in Cypher 25
section, the deprecated bit should probably be in aDeprecated features
section (not related to Cypher 25) and then inRemoved in Cypher 25
for the breaking change part 🤔