Skip to content

Conversation

JPryce-Aklundh
Copy link
Collaborator

@JPryce-Aklundh JPryce-Aklundh commented Jun 25, 2025

This PR will fail for now because it uses xrefs pointing to unmerged changes (part of #1299)

See also:
#1299

TODO:

  • Confirm release version (and update all instances of "2025.xx" in this PR to whatever that version is)
  • Port the following to cypher 5

    It is possible to drop constraints created in a different Cypher version than the one in use.
    For example, although VECTOR property type constraints were added in Cypher 25, such constraints can still be dropped using Cypher 5.

IMPORTANT: The changes made to managing-constraints.adoc must be ported over to constraints/create-constraints.adoc, constraints/list-constraints.adoc and schema/syntax.adoc (in #1395) once this PR has been merged (assuming vector type constraints go live before graph types)

Copy link
Collaborator

@Hunterness Hunterness left a comment

Choose a reason for hiding this comment

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

Did another look-through and found some more things

.Create `VECTOR` property type constraints label:cypher[Cypher 25 only] label:new[Introduced in Neo4j 2025.10]
======
It is necessary to specify both the dimension and the coordinate type of any constrained `VECTOR` properties.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
It is necessary to specify both the dimension and the coordinate type of any constrained `VECTOR` properties.
It is necessary to specify both the dimension and the coordinate type of all constrained `VECTOR` properties.

or

Suggested change
It is necessary to specify both the dimension and the coordinate type of any constrained `VECTOR` properties.
It is necessary to specify both the dimension and the coordinate type of any constrained `VECTOR` property.

maybe? sounded odd to me with any ... properties

* `ZONED DATETIME`
* `DURATION`
* `POINT`
* `VECTOR<TYPE>(DIMENSION)` label:cypher[Cypher 25 only] label:new[Introduced in Neo4j 2025.10]
Copy link
Collaborator

Choose a reason for hiding this comment

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

should we say what TYPE and DIMENSION is here? it's mentioned on the syntax page in the note below, and exists up by the example of vector property type constraints but not in this list of allowed types

stefano-ottolenghi added a commit that referenced this pull request Oct 9, 2025
See also: #1325

Co-authored-by: Stefano Ottolenghi <[email protected]>
@neo4j-docops-agent
Copy link
Collaborator

neo4j-docops-agent commented Oct 9, 2025

Thanks for the documentation updates.

The preview documentation has now been torn down - reopening this PR will republish it.

@stefano-ottolenghi stefano-ottolenghi marked this pull request as ready for review October 9, 2025 07:16
@stefano-ottolenghi stefano-ottolenghi merged commit 73baaa0 into neo4j:dev Oct 9, 2025
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants