-
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
QLTransactionTest.IntentsCleanupAfterRestart fails in master #1102
Labels
kind/bug
This issue is a bug
Comments
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
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
No description provided.
The text was updated successfully, but these errors were encountered: