-
Notifications
You must be signed in to change notification settings - Fork 58
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
Added DROP DATABASE name CASCADE ALIASES
#1036
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 |
---|---|---|
|
@@ -103,8 +103,21 @@ label:new[] | |
DROP [COMPOSITE] DATABASE ... [RESTRICT \| CASCADE ALIAS[ES]] | ||
---- | ||
|
||
| Added the ability to drop any database aliases, that would otherwise block the deletion of the database, at the same time as the deletion of the database. | ||
| Added the ability to drop database aliases while deleting a database. | ||
This will affect local database aliases targeting the database and constituent database aliases belonging to the composite database. | ||
For more information, see link:{neo4j-docs-base-uri}/operations-manual/{page-version}/database-administration/standard-databases/delete-databases/#delete-databases-with-aliases[Delete a database with local database aliases targeting it] and link:{neo4j-docs-base-uri}/operations-manual/{page-version}/database-administration/composite-databases/delete-composite-databases/#composite-databases-delete-with-aliases[Delete a composite database with constituent database aliases]. | ||
|
||
a| | ||
label:functionality[] | ||
label:new[] | ||
[source, cypher, role=noheader] | ||
---- | ||
LOAD CSV FROM 'azb://azb-account/azb-container/artists.csv' AS row | ||
MERGE (a:Artist {name: row[1], year: toInteger(row[2])}) | ||
RETURN a.name, a.year | ||
---- | ||
|
||
| Extension of the xref:clauses/load-csv.adoc#azure-cloud-storage[LOAD CSV] clause to allow loading CSV files from Azure Cloud Storage URIs. | ||
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. feels weird that this is added as part of this PR... 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. It wasnt - it was added here #1008 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. because you pull-merged instead of rebased XD |
||
|
||
|=== | ||
|
||
|
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.
maybe?
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.
though I guess we are already pretty inconsistent if we write drop or delete 🤷
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.
You are right - we need to decide on which to use and stick with it. Personally I think we should use drop because that is what the command is called (otherwise I definitely would have preferred "delete"). I will bring this up with my Ops Manual colleagues and take a decision.