-
Notifications
You must be signed in to change notification settings - Fork 848
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
Reduce transparent_decompress_chunk test flakiness #6518
Reduce transparent_decompress_chunk test flakiness #6518
Conversation
@@ -24,12 +22,12 @@ ALTER TABLE metrics DROP COLUMN filler_2; | |||
INSERT INTO metrics(time,device_id,v0,v1,v2,v3) SELECT time, device_id, device_id-1, device_id + 2, device_id + 0.5, NULL FROM generate_series('2000-01-06 0:00:00+0'::timestamptz,'2000-01-12 23:55:00+0','2m') gtime(time), generate_series(1,5,1) gdevice(device_id); | |||
ALTER TABLE metrics DROP COLUMN filler_3; | |||
INSERT INTO metrics(time,device_id,v0,v1,v2,v3) SELECT time, device_id, device_id, device_id + 2, device_id + 0.5, NULL FROM generate_series('2000-01-13 0:00:00+0'::timestamptz,'2000-01-19 23:55:00+0','2m') gtime(time), generate_series(1,5,1) gdevice(device_id); | |||
CREATE INDEX ON metrics(time DESC); |
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.
These are just to speed up shared setup.
One of the merge join queries is known to switch inner and outer join relations because the costs seem to be the same. Adding an additional filter on one of the relations should change the costs enough so they are not interchangeable.
18968aa
to
3975b69
Compare
@nikkhils, @erimatnor: please review this pull request.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6518 +/- ##
==========================================
- Coverage 87.33% 87.32% -0.01%
==========================================
Files 187 187
Lines 41899 41853 -46
Branches 9335 9315 -20
==========================================
- Hits 36593 36549 -44
+ Misses 3635 3627 -8
- Partials 1671 1677 +6 ☔ View full report in Codecov by Sentry. |
One of the merge join queries is known to switch inner and outer join relations because the costs seem to be the same. Adding an additional filter on one of the relations should change the costs enough so they are not interchangeable.
Fixes #5424
Disable-check: force-changelog-file