Skip to content

Revert "Disable some flappy tests"#8840

Merged
alesapin merged 3 commits intoClickHouse:masterfrom
excitoon-favorites:returnttltests
May 25, 2020
Merged

Revert "Disable some flappy tests"#8840
alesapin merged 3 commits intoClickHouse:masterfrom
excitoon-favorites:returnttltests

Conversation

@excitoon
Copy link
Copy Markdown
Contributor

This reverts commit e6958c0.

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Non-significant (changelog entry is not required)

@excitoon excitoon marked this pull request as ready for review January 30, 2020 23:58
@alexey-milovidov
Copy link
Copy Markdown
Member

alexey-milovidov commented Feb 2, 2020

@excitoon Please allow edits for your branch.

@alexey-milovidov
Copy link
Copy Markdown
Member

alexey-milovidov commented Feb 2, 2020

Still flapping.

@alexey-milovidov
Copy link
Copy Markdown
Member

Still failed.

@alexey-milovidov
Copy link
Copy Markdown
Member

?

@alexey-milovidov
Copy link
Copy Markdown
Member

Still failing.

@excitoon
Copy link
Copy Markdown
Contributor Author

excitoon commented May 11, 2020

I expect these test to start pass after #10588

@alexey-milovidov
Copy link
Copy Markdown
Member

They did not.

Copy link
Copy Markdown
Member

@alesapin alesapin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok

@alesapin alesapin merged commit 73458c1 into ClickHouse:master May 25, 2020
alexey-milovidov added a commit that referenced this pull request May 31, 2020
alexey-milovidov added a commit that referenced this pull request May 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-docs-needed pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants