-
Notifications
You must be signed in to change notification settings - Fork 83
Update the load command description as it supports full backups
#1594
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
Conversation
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.
Looks good. I added some minor suggestions.
|
|
||
| A database backup artifact (full or differential) can be restored within the same or to a later Neo4j version using the `restore` command of `neo4j-admin`. | ||
|
|
||
| Starting with Neo4j 5.20, you can load a **full** database backup artifact using the xref:backup-restore/restore-dump.adoc#restore-dump-command[`neo4j-admin database load`] command. |
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.
I would leave this command out of this page, but we can discuss it.
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.
my idea was to represent this new feature here as well because if you use EE and backups you don't need to go to the Restore dump page and you won't be aware of this new functionality.
|
|
||
| [source,shell, role="nocopy"] | ||
| ---- | ||
| bin/neo4j-admin database load --from-path=/full-path/to/backups <database> --overwrite-destination=true |
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.
I wonder if the path shouldn't point to the artifact as it does in the neo4j-admin database restore command.
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.
tbh, I think it's redundant in this case
Co-authored-by: Reneta Popova <reneta.popova@neo4j.com>
|
Thanks for the documentation updates. The preview documentation has now been torn down - reopening this PR will republish it. |
…o4j#1594) Co-authored-by: Reneta Popova <reneta.popova@neo4j.com>
No description provided.