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

QLTransactionTest.Child fails in master release #2407

Closed
spolitov opened this issue Sep 24, 2019 · 0 comments
Closed

QLTransactionTest.Child fails in master release #2407

spolitov opened this issue Sep 24, 2019 · 0 comments
Assignees
@spolitov
Labels
area/docdb YugabyteDB core features

Comments

@spolitov
Copy link
Contributor

No description provided.

@spolitov spolitov added the area/docdb YugabyteDB core features label Sep 24, 2019
@spolitov spolitov self-assigned this Sep 24, 2019
spolitov added a commit that referenced this issue Sep 24, 2019
@spolitov
[#2407]: Update peer list while checking for running transactions in …
a19da3c 
…tests

Summary:
At the end of some tests we check that all transactions were released.
For each tserver tablet peers are taken and checked for number of running transactions.
Before checking for number of running transactions we wait until tablet is created for particular peer.
But there are scenarios when peer is deleted and recreated (for instance when remote bootstrap is required).

But we still holding pointer to the old peer, which will never create a tablet.

Fixed by updating peer list.

Test Plan: ybd release --cxx-test client_ql-transaction-test --gtest_filter QLTransactionTest.Child --clang -n 100

Reviewers: timur, mikhail

Reviewed By: mikhail

Subscribers: ybase

Differential Revision: https://phabricator.dev.yugabyte.com/D7267
@snyk-bot snyk-bot mentioned this issue Jul 29, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@spolitov spolitov

Labels
area/docdb YugabyteDB core features
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@spolitov