Skip to content

Update Query routing decisions page#3171

Open
renetapopova wants to merge 3 commits into
neo4j:devfrom
renetapopova:dev-query-routing-decisions
Open

Update Query routing decisions page#3171
renetapopova wants to merge 3 commits into
neo4j:devfrom
renetapopova:dev-query-routing-decisions

Conversation

@renetapopova

Copy link
Copy Markdown
Collaborator

No description provided.

@renetapopova renetapopova added cherry-pick-this-to-5.x cherry-pick-this-to-main team-cypher-operations Cypher operations should review this and removed team-cypher-operations Cypher operations should review this labels Jul 9, 2026
@NataliaIvakina NataliaIvakina self-assigned this Jul 10, 2026

@ConorNeo ConorNeo left a comment

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.

Nice clear image!

@renetapopova

Copy link
Copy Markdown
Collaborator Author

Nice clear image!

Thanks for the review, @ConorNeo. Actually, the image is the only thing I didn't change, but looking at it now, I think it needs a bit of updating. It talks about the server leaders instead of databases, and also has just one READ possibility, where there should be two.

@ConorNeo

Copy link
Copy Markdown
Collaborator

Nice clear image!

Thanks for the review, @ConorNeo. Actually, the image is the only thing I didn't change, but looking at it now, I think it needs a bit of updating. It talks about the server leaders instead of databases, and also has just one READ possibility, where there should be two.

I think I might be going crazy. I can see no mention of "server leaders" in the image? What is the second read possibility of which you talk? :)

@renetapopova

renetapopova commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator Author

Nice clear image!

Thanks for the review, @ConorNeo. Actually, the image is the only thing I didn't change, but looking at it now, I think it needs a bit of updating. It talks about the server leaders instead of databases, and also has just one READ possibility, where there should be two.

I think I might be going crazy. I can see no mention of "server leaders" in the image? What is the second read possibility of which you talk? :)

I mean, we say "Is the local member the leader for that database?" but servers can't be leaders anymore; one of the databases is the leader for the rest of the same databases.

What is the second read possibility of which you talk? :)

Screenshot 2026-07-10 at 14 51 08

@renetapopova

Copy link
Copy Markdown
Collaborator Author

Hey @ConorNeo, could you please review the diagram? I have updated it according to the described steps, and if you are happy with it, I'll ask Lidia to reproduce it in the Neo4j style and colours.

@renetapopova
renetapopova requested a review from ConorNeo July 13, 2026 16:41

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.

If we want to update the image, should we add also 'Target database is a sharded database' to the step 3?

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.

I am not sure I understand. There are no sharded databases here. Do you mean to also add what the query routing decisions would be if the target database is an SPD?

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.

Maybe @ConorNeo can chime in here. I have no idea what the differences would be. We'll also need to update the steps.

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.

Do you mean to also add what the query routing decisions would be if the target database is an SPD?

yes, that's what I meant 😄 To add information about query routing in an SPD. But only if we want it. This is just a question.

Comment thread modules/ROOT/pages/database-administration/routing-decisions.adoc Outdated
Co-authored-by: Natalia Ivakina <82437520+NataliaIvakina@users.noreply.github.com>
@neo4j-docops-agent

Copy link
Copy Markdown
Collaborator

This PR includes documentation updates
View the updated docs at https://neo4j-docs-operations-3171.surge.sh

Updated pages:

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.

4 participants