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
Randomize more settings #39663
Randomize more settings #39663
Conversation
tests/clickhouse-test
Outdated
return ( | ||
lambda: 0 | ||
if random.random() < 0.5 | ||
else 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i'm afraid that it may slow down some tests significantly. have you checked?
@CurtizJ, this is so good pull request; let's continue! |
We have faced a bug on GitHub. |
@CurtizJ This is so wonderful, let's continue! |
This is an automated comment for commit dbabaeb with description of existing statuses. It's updated for the latest CI running ❌ Click here to open a full report in a separate page Successful checks
|
We need this. |
@CurtizJ, We still need this. |
We need this. |
@CurtizJ We need this PR to be merged. |
@CurtizJ This PR makes sense. |
How nice it is. |
@CurtizJ We need this. |
This pull request is good to continue. |
This is a good pull request - let's continue. |
We need this pull request to be merged. |
@CurtizJ, please continue 😸 |
bc4f41d
to
e5f849b
Compare
Current (not complete) list of failed tests:
|
|
|
|
b8cc655
to
30b09f4
Compare
|
So I found 4 real issues in this PR: #57686, #57688, #57692, #57814 Two of them (#57686 and #57692) are fixed in #57795 and #57790. #57688 is closed with #57814 looks a bit weird. Probably should be closed with |
0568eaf
to
4ff6d47
Compare
Last failing test:
|
|
Tests look ok except |
Anyway there will be some new flaky tests. |
It became flaky after we merged #39663. When external aggregation happens it seems like we get more than one part, and because we used `OPTIMIZE` without `FINAL` data wasn't properly merged. `order by` on the line 12 is not needed by now, but let's add it just in case.
It became flaky after we merged ClickHouse#39663. When external aggregation happens it seems like we get more than one part, and because we used `OPTIMIZE` without `FINAL` data wasn't properly merged. `order by` on the line 12 is not needed by now, but let's add it just in case.
Changelog category (leave one):