Skip to content

Conversation

@lidiazuin
Copy link
Contributor

No description provided.

Copy link
Contributor

@AlexicaWright AlexicaWright left a comment

Choose a reason for hiding this comment

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

Very useful information! I have a few comments, but also what about the Import service in Aura? That allows you to import from PostgreSQL, MySQL, SQL Server, Oracle, and Snowflake in addition to local CSV.

^.^| Description
| Available on Aura
| Available on self-managed
| Enterprise only
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this column needed? Maybe this information can be incorporated into the previous one? Enterprise also applies to AuraDS.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't know if I understand what you mean by incorporating the 'Enterprise only' column to 'Available on self-managed' only?

Copy link
Contributor

Choose a reason for hiding this comment

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

If this column refers to the Enterprise Edition of self-managed Neo4j, maybe it can be shown in that column instead? But there is also an Enterprise tier for AuraDS, so saying Enterprise only can be confusing. Also, it feels excluding. Would it be possible to maybe add a footnote (in the self-managed column) to the Import options that are available only on EE (if that's what is meant by Enterprise)?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I updated the PR with what we discussed, after investigating :)


a| link:https://www.neo4j.com/docs/operations-manual/current/import/#import-tool-full[`neo4j-admin database import full`]
| Initial import into a non-existent empty database.
Use it with link:{docs-home}/operations-manual/current/neo4j-admin-neo4j-cli/[Neo4j Admin].
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe this can be rephrased? The command is a neo4j-admin command so it sounds a bit weird to say to use it with the tool.

a| link:https://www.neo4j.com/docs/operations-manual/current/import/#import-tool-incremental[`neo4j-admin database import incremental`]
| Used when import cannot be completed in a single full process.
It allows the import to be performed as a series of smaller batches.
Use it with link:{docs-home}/operations-manual/current/neo4j-admin-neo4j-cli/[Neo4j Admin].
Copy link
Contributor

Choose a reason for hiding this comment

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

As above.

Copy link
Contributor

@AlexicaWright AlexicaWright left a comment

Choose a reason for hiding this comment

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

A few more comments.

@lidiazuin lidiazuin requested a review from AlexicaWright May 5, 2025 11:41
Copy link
Contributor

@AlexicaWright AlexicaWright left a comment

Choose a reason for hiding this comment

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

One more data source is now available. Otherwise it looks great! Thanks @lidiazuin !

@neo4j-docops-agent
Copy link
Collaborator

neo4j-docops-agent commented May 5, 2025

Thanks for the documentation updates.

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

@lidiazuin lidiazuin merged commit 93b97fa into neo4j:dev May 5, 2025
5 checks passed
@lidiazuin lidiazuin deleted the dataimport branch May 5, 2025 14:10
lidiazuin added a commit that referenced this pull request May 5, 2025
* cherry-pick

* fixing link
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.

3 participants