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
Add the Flaky Check #4883
Add the Flaky Check #4883
Conversation
Codecov Report
@@ Coverage Diff @@
## main #4883 +/- ##
==========================================
- Coverage 89.48% 89.44% -0.05%
==========================================
Files 225 225
Lines 50288 50287 -1
==========================================
- Hits 45001 44978 -23
- Misses 5287 5309 +22
Continue to review full report at Codecov.
|
"installcheck_args": "IGNORES='bgw_db_scheduler bgw_db_scheduler_fixed bgw_launcher pg_dump remote_connection compressed_collation'", | ||
"llvm_config": "/usr/local/opt/llvm/bin/llvm-config", | ||
"os": "macos-11", | ||
"pg": PG12_LATEST, |
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.
Should we still use PG12 here?
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 don't know... probably should remove support when we start supporting PG 15.
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.
The plan is remove it when we have full PG15 support!
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.
LGTM overall.
test/sql/agg_bookends.sql.in
Outdated
@@ -31,3 +31,5 @@ SET timescaledb.enable_optimizations TO true; | |||
\o | |||
|
|||
:DIFF_CMD | |||
|
|||
|
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.
Leftover?
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.
Yep, I used them to test that the check works.
It runs new or changed tests multiple times to find flakiness.
It runs the changed tests many times to find flakiness.