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.

Sign up for GitHub

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

Jump to bottom

[CDCSDK] Message bus integration of CDC (PubSub, Kinesis, etc) #11861

Closed
vaibhav-yb opened this issue Mar 23, 2022 · 0 comments
Closed

[CDCSDK] Message bus integration of CDC (PubSub, Kinesis, etc) #11861

vaibhav-yb opened this issue Mar 23, 2022 · 0 comments
Assignees
@suranjan
Labels
area/cdcsdk CDC SDK kind/bug This issue is a bug priority/medium Medium priority issue

Comments

@vaibhav-yb
Copy link
Contributor

vaibhav-yb commented Mar 23, 2022 •
edited by yugabyte-ci
Loading

Jira Link: DB-4519
@vaibhav-yb vaibhav-yb added the area/cdcsdk CDC SDK label Mar 23, 2022
@vaibhav-yb vaibhav-yb mentioned this issue Mar 24, 2022
Open
vaibhav-yb added a commit that referenced this issue Apr 11, 2022
@vaibhav-yb
[#11861][CDCSDK] Fix for the flaky test CDCSDKYsqlTest.TestNeedSchema…
cba240b 
…InfoFlag

Summary:
This fix to aims to address the intermittent failure observed in the corresponding test. The issue was that with RF 3, if we call `GetChanges` with the `need_schema_info` flag set to `false` and if the request goes to one of the follower nodes, plus if it's the first time the follower node is serving the request we will get a DDL record due to the way the test is structured, and it would lead to a failure of the assertion where we are expecting that we should not get a DDL record.

To address that, the replication factor of the server is changed to 1 which, in turn, will result in a single node, so all the requests will be served by it.

Test Plan:
* `ybd --cxx-test cdcsdk_ysql-test --gtest_filter CDCSDKYsqlTest.TestNeedSchemaInfoFlag`
* `ybd --cxx-test cdcsdk_ysql-test --gtest_filter CDCSDKYsqlTest.TestNeedSchemaInfoFlag -n 50`

Reviewers: sdash, iamoncar, skumar

Reviewed By: skumar

Subscribers: ybase

Differential Revision: https://phabricator.dev.yugabyte.com/D16423
@yugabyte-ci yugabyte-ci added kind/bug This issue is a bug priority/medium Medium priority issue labels Dec 20, 2022
@ryan-ally ryan-ally mentioned this issue Nov 30, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@suranjan suranjan

Labels
area/cdcsdk CDC SDK kind/bug This issue is a bug priority/medium Medium priority issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@suranjan @yugabyte-ci @vaibhav-yb