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

Travis: Enable retries for unit tests as well. #921

Merged

Conversation

michael-berlin
Copy link
Contributor

At this point, they are too flaky and it hinders productivity too much to bother with that.

@alainjobart

At this point, they are too flaky and it hinders productivity too much to bother with that.
@alainjobart
Copy link
Contributor

LGTM

michael-berlin added a commit that referenced this pull request Jul 24, 2015
Travis: Enable retries for unit tests as well.
@michael-berlin michael-berlin merged commit eab9fad into vitessio:master Jul 24, 2015
@michael-berlin michael-berlin deleted the travis_retry_unit_tests branch July 24, 2015 23:23
systay pushed a commit to planetscale/vitess that referenced this pull request Aug 19, 2022
The refactor of the Watch implementation led me to accidentally adding
this to the watcher, but we should keep the same implementation and not
add those close here.

Signed-off-by: Dirkjan Bussink <d.bussink@gmail.com>
notfelineit pushed a commit to planetscale/vitess that referenced this pull request Sep 21, 2022
* Revert "Add explicit close state to memory topo connection (vitessio#11110) (vitessio#1016)"

This reverts commit eb1e9c2.

* Revert "Fix races in memory topo and watcher (vitessio#11065) (vitessio#995)"

This reverts commit 6bc0171.

* Revert "Avoid race condition in watch shutdown (vitessio#10954) (vitessio#936)"

This reverts commit 23d4e34.

* Revert "Remove potential double close of channel (vitessio#10929) (vitessio#921)"

This reverts commit 0121e5d.

* Revert "Cherry pick topo improvements (vitessio#10906) (vitessio#916)"

This reverts commit 8c9f56d.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants