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
Move timestamp_limits and with_clause_parser test #3346
Conversation
9dfa3c3
to
f6c6119
Compare
@@ -6,6 +6,8 @@ SET client_min_messages TO ERROR; | |||
|
|||
SET ROLE :ROLE_DEFAULT_PERM_USER; | |||
|
|||
CREATE SCHEMA test; |
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.
why are we adding this?
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.
timestamp_limits test uses this and on normal regresscheck we precreate that schema too
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.
changes look good. Question about a test change.
Move the timestamp_limits and with_clause_parser test to regresscheck-shared since those tests don't need a private database incurring less overhead to run those tests. Also add missing ORDER BY clauses to some of the queries in timestamp_limits to make the output more stable.
Codecov Report
@@ Coverage Diff @@
## master #3346 +/- ##
==========================================
- Coverage 92.17% 90.58% -1.59%
==========================================
Files 110 212 +102
Lines 14177 35589 +21412
==========================================
+ Hits 13067 32240 +19173
- Misses 1110 3349 +2239
Continue to review full report at Codecov.
|
Move the timestamp_limits and with_clause_parser test to
regresscheck-shared since those tests don't need a private
database incurring less overhead to run those tests.