Skip to content

Conversation

ravikiran-sulikeri
Copy link
Contributor

ravikiran-sulikeri and others added 2 commits October 3, 2024 09:53
Co-authored-by: Jessica Wright <49636617+AlexicaWright@users.noreply.github.com>
Copy link
Collaborator

@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.

LGTM! However, the content about secondary instances has not yet been merged into the aura docs, #396.

@ravikiran-sulikeri
Copy link
Contributor Author

LGTM! However, the content about secondary instances has not yet been merged into the aura docs, #396.

I will just remove the references to secondaries from the text for now.

Copy link
Collaborator

@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.

Is clustering in Aura surfaced to the user?

@ravikiran-sulikeri
Copy link
Contributor Author

Is clustering in Aura surfaced to the user?

Not entirely but these metric values are presented to the user over availability zone groupings which doesn't really reveal anything about Aura clustering other than the existence of cluster members.

@AlexicaWright
Copy link
Collaborator

I assume that the secondaries will go live shortly, so with the omission of secondaries in this PR, I'd say let's merge.

@ravikiran-sulikeri
Copy link
Contributor Author

I assume that the secondaries will go live shortly, so with the omission of secondaries in this PR, I'd say let's merge.

I am still waiting for other code change PRs to be merged, I will let you know when they are ready.

@ravikiran-sulikeri
Copy link
Contributor Author

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

neo-technology-commit-status-publisher commented Oct 8, 2024

Thanks for the documentation updates.

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

@AlexicaWright AlexicaWright merged commit 5db95a9 into neo4j:main Oct 10, 2024
4 checks passed
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.

3 participants