Conversation
We're allowing CALL procedure both together with SHOW/TERMINATE commands and multiple CALL clauses together so 42NA9 is out of date and should be updated. Also add examples for both that and 42N17.
Hunterness
added
DO NOT MERGE
team-cypher-operations
renetapopova
left a comment
renetapopova
left a comment
There was a problem hiding this comment.
Hey @Hunterness, thanks for the examples. Really useful. I left some editorial comments, mainly related to consistency with the rest of the errors.
Co-authored-by: Reneta Popova <reneta.popova@neo4j.com>
|
This PR includes documentation updates Updated pages: |
renetapopova
left a comment
renetapopova
left a comment
There was a problem hiding this comment.
Should be good now. Could you please take a final look?
There was a problem hiding this comment.
Looks good, but I'll keep the DO NOT MERGE label and it not being merged until I merge the code updates for the 42AN9 code (one of my other tasks came back and took priority so I didn't get around to finishing the coding side yet :( hopefully next week 🤞)
Since 42AN9 is an existing code in use I think we should keep the existing message until it's actually changed
3 participants
We're allowing CALL procedure both together with (some of the) SHOW/TERMINATE commands and multiple CALL clauses together so 42NA9 is out of date and should be updated.
Also add examples for both that and 42N17.
The updates to behaviour have not yet been merged, so neither should this PR.