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

add line about prerequisite for database upload #375

Merged
merged 2 commits into from
Jun 14, 2024

Conversation

AlexicaWright
Copy link
Collaborator

No description provided.

@neo-technology-commit-status-publisher
Copy link
Collaborator

Thanks for the documentation updates.

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

@@ -28,6 +28,7 @@ See <<_public_traffic>> below for more information.

`database upload` is a `neo4j-admin` command that you can run to upload the contents of a Neo4j database into an Aura instance, regardless of the database's size.
Keep in mind that the database you want to upload may run a different version of Neo4j than your Aura instance.
Additionally, the machine you run the `database upload` command from must have access to the Aura instance you want to upload the database to.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Additionally, the machine you run the `database upload` command from must have access to the Aura instance you want to upload the database to.
Additionally, the machine from which you run the `database upload` command must have access to the Aura instance to which you want to upload the database.

or similar to the Operation manual:

Additionally, your Neo4j Aura instance must be accessible from the machine running `neo4j-admin`. 
Otherwise, the upload will fail with SSL errors.

Copy link
Contributor

@renetapopova renetapopova left a comment

Choose a reason for hiding this comment

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

I added two suggestions. I like the second one more if you think it will be consistent with the rest of the content on this page.

Copy link
Contributor

@renetapopova renetapopova left a comment

Choose a reason for hiding this comment

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

Looks perfect 😄

@AlexicaWright AlexicaWright merged commit 7857a50 into neo4j:main Jun 14, 2024
4 checks passed
@fiquick
Copy link
Contributor

fiquick commented Jun 25, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants