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

Docs-1003 : docs expand subscriber only node content #6112

Conversation

djw-m
Copy link
Contributor

@djw-m djw-m commented Sep 30, 2024

What Changed?

Added content for subscriber-only and himesh implementation.

josh-heyer and others added 14 commits September 18, 2024 17:27
Signed-off-by: Dj Walker-Morgan <[email protected]>
Signed-off-by: Dj Walker-Morgan <[email protected]>
From PGD 5.6, database structure will never be synchronized to
a witness server, which represents a minor behaviour change compared
to previous versions, and means the "synchronize_structure" parameter,
if explicitly set to "all", will be forcibly overridden (a NOTICE will
be emitted noting that).

DOCS-1029.
Signed-off-by: Dj Walker-Morgan <[email protected]>
Signed-off-by: Dj Walker-Morgan <[email protected]>
Signed-off-by: Dj Walker-Morgan <[email protected]>
@djw-m djw-m requested a review from a team as a code owner September 30, 2024 08:16
josh-heyer and others added 11 commits September 30, 2024 16:23
Signed-off-by: Dj Walker-Morgan <[email protected]>
Signed-off-by: Dj Walker-Morgan <[email protected]>
From PGD 5.6, database structure will never be synchronized to
a witness server, which represents a minor behaviour change compared
to previous versions, and means the "synchronize_structure" parameter,
if explicitly set to "all", will be forcibly overridden (a NOTICE will
be emitted noting that).

DOCS-1029.
* Complete changes, table formatted all parameters, fixed phrasing

Signed-off-by: Dj Walker-Morgan <[email protected]>

* Adjusted text as groups no longer shown

Signed-off-by: Dj Walker-Morgan <[email protected]>

---------

Signed-off-by: Dj Walker-Morgan <[email protected]>
@djw-m djw-m changed the title Docs 1003 docs expand subscriber only node content Docs-1003 : docs expand subscriber only node content Sep 30, 2024
Signed-off-by: Dj Walker-Morgan <[email protected]>
Signed-off-by: Dj Walker-Morgan <[email protected]>
@djw-m djw-m added the deploy Add this label to a PR and it will automatically be deployed to netlify label Oct 2, 2024
Copy link
Contributor

github-actions bot commented Oct 2, 2024

Copy link
Contributor

@Metaluim Metaluim left a comment

Choose a reason for hiding this comment

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

Hi @djw-m ,

Great job explaining the changes to SO groups. I just have a couple of remarks.


* Data-nodes which are directly members of the top-level group.
* No data-node sub-groups.
* No data-node sub-groups with proxy routing enabled.
Copy link
Contributor

Choose a reason for hiding this comment

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

Consider linking here a page regarding this setting.

@djw-m
Copy link
Contributor Author

djw-m commented Oct 3, 2024

Due to Github being awful, this PR is closed and a new #6125 PR is open for comments, with all the comments here being handled in it. Apologies.

@djw-m djw-m closed this Oct 3, 2024

## Enabling the optimized model

By default, PGD 5.6 will force the full mesh topology. To enable the himesh topology, you must have your data-nodes in subgroups, with proxy routing enabled on the sub groups. Then you can set `bdr.force_full_mesh` to `false` to allow the optimization to take effect.
Copy link
Contributor

@sumarjee sumarjee Oct 4, 2024

Choose a reason for hiding this comment

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

We may want to mention the change in this parameter requires a server restart to take effect.


This group leader acts as a replication proxy for incoming changes.

This model cannot be enabled if a cluster has any of the following:
Copy link
Contributor

Choose a reason for hiding this comment

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

Will a positive tone be better? Like say Following are pre-requisites to enable this model.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy Add this label to a PR and it will automatically be deployed to netlify
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants