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

[doc][cdc] Updated diagrams #23262

Merged
merged 2 commits into from
Jul 24, 2024
Merged

[doc][cdc] Updated diagrams #23262

merged 2 commits into from
Jul 24, 2024

Conversation

ddhodge
Copy link
Contributor

@ddhodge ddhodge commented Jul 23, 2024

Updated diagrams

@netlify /preview/architecture/docdb-replication/change-data-capture/

@ddhodge ddhodge self-assigned this Jul 23, 2024
@ddhodge ddhodge added the area/documentation Documentation needed label Jul 23, 2024
Copy link

netlify bot commented Jul 23, 2024

Deploy Preview for infallible-bardeen-164bc9 ready!

Name Link
🔨 Latest commit 0269fc6
🔍 Latest deploy log https://app.netlify.com/sites/infallible-bardeen-164bc9/deploys/66a159536a5adc00089aa9bc
😎 Deploy Preview https://deploy-preview-23262--infallible-bardeen-164bc9.netlify.app/preview/architecture/docdb-replication/change-data-capture/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@siddharth2411
Copy link
Contributor

siddharth2411 commented Jul 23, 2024

Reviewed the architecture diagram, it looks good to me. Would defer to @ymahajan for approval since there are other changes.

Copy link
Contributor

@ymahajan ymahajan left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@ddhodge ddhodge requested a review from aishwarya24 July 24, 2024 17:22
Copy link
Collaborator

@aishwarya24 aishwarya24 left a comment

Choose a reason for hiding this comment

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

LGTM

Co-authored-by: Aishwarya Chakravarthy  <achakravarthy@yugabyte.com>
@ddhodge ddhodge merged commit 1b3585f into yugabyte:master Jul 24, 2024
2 checks passed
@ddhodge ddhodge deleted the doc_cdc_diagrams branch July 24, 2024 20:04
jasonyb pushed a commit that referenced this pull request Jul 25, 2024
Summary:
 5aa0c0a [PLAT-14078] Add local provider test for update databases
 cdd97f8 remove ea badge (#23276)
 2813d78 [PLAT-14156][PLAT-14323]: Move all UI Driven flags to INTERNAL and remove YBM key as its not used
 49523f5 [PLAT-14733]: Add support for OIDC attributes jwt_jwks_path and jwt_jwks_url
 b039d1a [PLAT-14366] Basic local provider test for master auto failover
 700fd49 [#23275] docdb: Fix missing home icon on master UI
 89e434e [#13254] YSQL: import pgtap v1.3.3
 1b3585f [doc][cdc] Updated diagrams (#23262)
 399f165 [#23266] YSQL: Only require YB Admin privileges to run pg_locks
 5a4bbd4 [#19954] docdb: Register both tablet split children atomically
 b4c4294 [PLAT-14617] Add support for numerical search and enable extra search fields forxCluster
 84fb7ad [#22449] YSQL: wal2json YB specific changes
 afe84d4 [#13254] YSQL: add pgtap to build
 adf3c54 [#23272] YSQL, ASH: Fix incorrect popping of query id from nested query ids stack
 3b42c2e [docs] Add syntax documentation for logical replication (#23270)

Test Plan: Jenkins: rebase: pg15-cherrypicks

Reviewers: jason, tfoucher

Tags: #jenkins-ready

Differential Revision: https://phorge.dev.yugabyte.com/D36841
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Documentation needed
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

4 participants