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

Specify Cypher 9 #261

Merged
merged 2 commits into from Oct 16, 2018

Conversation

Projects
None yet
2 participants
@Mats-SX
Copy link
Member

commented Oct 17, 2017

Outlines a proposal for Cypher 9 -- the first official standard Cypher version as governed by the openCypher project.

The specification is made up of three resources: the Neo4j 3.2 Manual, the Standardisation Scope document, and this CIP. Together these three components define Cypher 9.

Mats-SX added some commits Sep 29, 2017

Specify Cypher 9
- Archive three old CIPs
- Document archive state of CIPs

Modify Standardisation Scope according to CLG discussion.

@Mats-SX Mats-SX force-pushed the Mats-SX:cypher-9 branch from 41678b0 to 7e78a2a Oct 17, 2017

@petraselmer
Copy link
Contributor

left a comment

Looks great! Just a couple of small comments


New syntax for these constructs needs to be proposed to consider them applicable for the standard.

The `exists()` function is defined in an accepted CIP, and does not need to rely on vendor documentation.

This comment has been minimized.

Copy link
@petraselmer

petraselmer Oct 18, 2017

Contributor

Aren't we including the form exists(n.prop) and excluding the form exists(pattern)? This sentence is not very clear.

This comment has been minimized.

Copy link
@Mats-SX

Mats-SX Oct 19, 2017

Author Member

Not as documented in the Neo4j manual, but as specified in the CIP. I thought a CIP is better than referring to the manual?

This comment has been minimized.

Copy link
@Mats-SX

Mats-SX Oct 19, 2017

Author Member

My interpretation on the Standardisation Scope is that it only acts as an overlay of what's in Neo4j ('s manual), and doesn't reference anything else.

* `any()`
* `distance()`
* `exists()`

This comment has been minimized.

Copy link
@petraselmer

petraselmer Oct 18, 2017

Contributor

Please see my earlier comment about the two forms of exists()

This comment has been minimized.

Copy link
@Mats-SX

Mats-SX Oct 19, 2017

Author Member

It's not included as implemented in Neo4j, but it is included as specified in the EXISTS CIP.

@petraselmer petraselmer added the CIP label Oct 20, 2017

@Mats-SX

This comment has been minimized.

Copy link
Member Author

commented Oct 16, 2018

@petraselmer Do you think this is obsoleted in the presence of the Cypher 9 reference manual on opencypher.org?

@petraselmer

This comment has been minimized.

Copy link
Contributor

commented Oct 16, 2018

@Mats-SX I must say I have found this very useful. At a glance, one can see very easily what's in and what's "out", which is obscured in the manual. So, I would say it is well worth having this as a 'quick reference' type of resource.

@Mats-SX

This comment has been minimized.

Copy link
Member Author

commented Oct 16, 2018

Then I suggest we merge it :)

@petraselmer petraselmer merged commit 1db001d into opencypher:master Oct 16, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.