Skip to content

Add a page about resharding a database and a few other improvements#2960

Merged
renetapopova merged 29 commits intoneo4j:devfrom
renetapopova:spd-resharding
Apr 2, 2026
Merged

Add a page about resharding a database and a few other improvements#2960
renetapopova merged 29 commits intoneo4j:devfrom
renetapopova:spd-resharding

Conversation

@renetapopova
Copy link
Copy Markdown
Collaborator

This PR contains the following changes to pages:

  1. Moved the examples of how to create a sharded property database from the Data ingestion page to Creating sharded property databases.
  2. Renamed the page Data ingestion to Data import
  3. Created a new page called Resharding a database.
  • Moved the section Resizing and resharding from Admin operations to the new page
  • Moved Split an existing database into shards (offline)
  • Add information about creating a sharded property database from a backup of a standard database and a backup of a sharded property database.

Copy link
Copy Markdown
Collaborator

@EmelieBondesson EmelieBondesson 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 small comments but apart from that it looks good to me

@jackwaudby jackwaudby self-assigned this Mar 26, 2026
@renetapopova
Copy link
Copy Markdown
Collaborator Author

The test failure is not related to this PR.

You can use the `neo4j-admin database copy` command to create a sharded property database from a standard database, specifying the number of property shards for the new database.
The existing database must be stopped during the copying process.

The following options should be specified for the `neo4j-admin database copy` command:
Copy link
Copy Markdown
Collaborator

@chamocarrot chamocarrot Apr 1, 2026

Choose a reason for hiding this comment

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

As they are not mandatory it should not say "should" but "can". Or maybe write:
The following options will be used for the neo4j-admin database copy command

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

But if you don't specify them, you can't split an existing database into shards.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This section is about this, not about the copy command per se.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

My comment comes from that:
--copy-schema: You can do it without it, and actually we are writing "if needed"
--target-location can be skipped and --target-format then skipped as well or set to database and then the database stores will be created in the neo4j store location.
This is cosmetic so we can take later discussions :)


There are several ways to load data into a sharded property database:
There are several ways to import data into a sharded property database, depending on your use case and the size of your dataset.
All of these methods require that the database is not running.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is conflicting with presenting LOAD CSV right after (it needs a running database). And neo4j-admin can be run with a neo4j already running. Only issue would be competing for resources (memory and cpu). Unless there is a very potent reason why this phrase was included maybe is better to remove it until we have a better wording.

Comment thread modules/ROOT/pages/scalability/sharded-property-databases/data-ingestion.adoc Outdated
Comment thread modules/ROOT/pages/scalability/sharded-property-databases/data-ingestion.adoc Outdated
Comment thread modules/ROOT/pages/scalability/sharded-property-databases/data-ingestion.adoc Outdated
Comment thread modules/ROOT/pages/scalability/sharded-property-databases/data-ingestion.adoc Outdated
Comment thread modules/ROOT/pages/scalability/sharded-property-databases/data-ingestion.adoc Outdated
Comment thread modules/ROOT/pages/scalability/sharded-property-databases/data-ingestion.adoc Outdated
Comment thread modules/ROOT/pages/scalability/sharded-property-databases/data-ingestion.adoc Outdated
@neo4j-docops-agent
Copy link
Copy Markdown
Collaborator

neo4j-docops-agent commented Apr 2, 2026

Thanks for the documentation updates.

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

@renetapopova renetapopova merged commit a84e6fa into neo4j:dev Apr 2, 2026
4 of 5 checks passed
@renetapopova renetapopova deleted the spd-resharding branch April 2, 2026 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants