-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[doc] CDC docs changes for 2024.2.2 release #25794
[doc] CDC docs changes for 2024.2.2 release #25794
Conversation
✅ Deploy Preview for infallible-bardeen-164bc9 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
8d87a4e
to
86e594d
Compare
docs/content/preview/develop/change-data-capture/using-logical-replication/_index.md
Outdated
Show resolved
Hide resolved
docs/content/preview/develop/change-data-capture/using-logical-replication/advanced-topic.md
Outdated
Show resolved
Hide resolved
docs/content/stable/develop/change-data-capture/using-logical-replication/_index.md
Outdated
Show resolved
Hide resolved
docs/content/stable/develop/change-data-capture/using-logical-replication/advanced-topic.md
Outdated
Show resolved
Hide resolved
…-replication/_index.md Co-authored-by: Vaibhav Kushwaha <[email protected]>
…-replication/advanced-topic.md Co-authored-by: Vaibhav Kushwaha <[email protected]>
Co-authored-by: Vaibhav Kushwaha <[email protected]>
docs/content/preview/develop/change-data-capture/using-logical-replication/_index.md
Outdated
Show resolved
Hide resolved
docs/content/preview/develop/change-data-capture/using-logical-replication/advanced-topic.md
Outdated
Show resolved
Hide resolved
docs/content/preview/develop/change-data-capture/using-logical-replication/advanced-topic.md
Outdated
Show resolved
Hide resolved
docs/content/preview/develop/change-data-capture/using-logical-replication/advanced-topic.md
Outdated
Show resolved
Hide resolved
docs/content/preview/develop/change-data-capture/using-logical-replication/advanced-topic.md
Outdated
Show resolved
Hide resolved
docs/content/preview/develop/change-data-capture/using-logical-replication/advanced-topic.md
Outdated
Show resolved
Hide resolved
docs/content/preview/develop/change-data-capture/using-logical-replication/advanced-topic.md
Outdated
Show resolved
Hide resolved
docs/content/preview/develop/change-data-capture/using-logical-replication/advanced-topic.md
Outdated
Show resolved
Hide resolved
``` | ||
|
||
{{< note title="Important" >}} | ||
In the event that a user ends up reducing the value of `cdcsdk_publication_list_refresh_interval_secs`, it is recommended to increase the value of the flag back to its original value once you start receiving changes from the new table. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For every publication refresh we incur overhead to invalidate the relcache. Reducing the interval for publication refresh would mean we incur this overhead much more frequently.
This PR contains the following changes to CDC documentation for 2024.2.2 release:
cdc_intent_retention_ms
andcdcsdk_tablet_not_of_interest_timeout_secs
are now both master as well as tserver flags. So here we add these flags to the master list as well.cdcsdk_publication_list_refresh_interval_secs
has been reduced to 900 seconds (15 mins) from 3600 seconds (1 hr). This change also required changing an example in documentation about the behaviour of dynamic tables in logical replication.cdcsdk_flush_lag
has been introduced for logical replication. In this PR we have added it to the list of CDC metrics