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

QLTransactionTest.IntentsCleanupAfterRestart fails in master #1102

Closed
spolitov opened this issue Apr 1, 2019 · 0 comments
Closed

QLTransactionTest.IntentsCleanupAfterRestart fails in master #1102

spolitov opened this issue Apr 1, 2019 · 0 comments
Assignees
Labels
kind/bug This issue is a bug

Comments

@spolitov
Copy link
Contributor

spolitov commented Apr 1, 2019

No description provided.

@spolitov spolitov added the kind/bug This issue is a bug label Apr 1, 2019
@spolitov spolitov self-assigned this Apr 1, 2019
yugabyte-ci pushed a commit that referenced this issue Apr 3, 2019
…start

Summary:
The following steps were done to improve stability of QLTransactionTest.IntentsCleanupAfterRestart:
1) Use public IP for web server in mini cluster. So it would not try to resolve FQDN, that takes a lot of time in case of failure.
2) Decrease required compacted bytes to 5Kb, since sometimes only 7-8Kb are compacted, even when test succeeds.

Also add/improve log prefix for several classes, so it have both tablet and peer UUIDs.

Test Plan: ybd debug --gtest_filter QLTransactionTest.IntentsCleanupAfterRestart -n 20

Reviewers: mikhail, alex, timur

Reviewed By: timur

Subscribers: bogdan, bharat, ybase

Differential Revision: https://phabricator.dev.yugabyte.com/D6419
@spolitov spolitov closed this as completed Apr 4, 2019
mbautin pushed a commit that referenced this issue Jul 11, 2019
…ed to the

earlier commit 90680ca

Original commit message:

[#1102]: Improve stability of QLTransactionTest.IntentsCleanupAfterRestart

Summary:
The following steps were done to improve stability of QLTransactionTest.IntentsCleanupAfterRestart:
1) Use public IP for web server in mini cluster. So it would not try to resolve FQDN, that takes a lot of time in case of failure.
2) Decrease required compacted bytes to 5Kb, since sometimes only 7-8Kb are compacted, even when test succeeds.

Also add/improve log prefix for several classes, so it have both tablet and peer UUIDs.

Test Plan: ybd debug --gtest_filter QLTransactionTest.IntentsCleanupAfterRestart -n 20

Reviewers: mikhail, alex, timur

Reviewed By: timur

Subscribers: bogdan, bharat, ybase

Differential Revision: https://phabricator.dev.yugabyte.com/D6419
mbautin pushed a commit to mbautin/yugabyte-db that referenced this issue Jul 16, 2019
…pAfterRestart

Summary:
The following steps were done to improve stability of QLTransactionTest.IntentsCleanupAfterRestart:
1) Use public IP for web server in mini cluster. So it would not try to resolve FQDN, that takes a lot of time in case of failure.
2) Decrease required compacted bytes to 5Kb, since sometimes only 7-8Kb are compacted, even when test succeeds.

Also add/improve log prefix for several classes, so it have both tablet and peer UUIDs.

Test Plan: ybd debug --gtest_filter QLTransactionTest.IntentsCleanupAfterRestart -n 20

Reviewers: mikhail, alex, timur

Reviewed By: timur

Subscribers: bogdan, bharat, ybase

Differential Revision: https://phabricator.dev.yugabyte.com/D6419

Note:
This commit provides additional functionality that is logically related to
the earlier commit yugabyte@90680ca
and supersedes the commit yugabyte@c2c1ccd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug This issue is a bug
Projects
None yet
Development

No branches or pull requests

1 participant