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 test for dimensions view #2419
Conversation
tsl/test/expected/dist_views.out
Outdated
@@ -39,15 +39,22 @@ SELECT setseed(1); | |||
|
|||
(1 row) | |||
|
|||
CREATE TABLE dist_table(time timestamptz, device int, temp float); | |||
CREATE TABLE dist_table(time timestamptz, device int, temp float, timdim date); |
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.
can you make time and timdim NOT NULL, this will get rid of the NOTICEs and remove some noise
16ebc27
to
bd016ce
Compare
tsl/test/expected/dist_views.out
Outdated
@@ -39,15 +39,20 @@ SELECT setseed(1); | |||
|
|||
(1 row) | |||
|
|||
CREATE TABLE dist_table(time timestamptz, device int, temp float); | |||
CREATE TABLE dist_table(time timestamptz NOT NULL, device int, temp float, timdim date NOT NULL); |
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.
did you mean to call that column timedim instead of timdim just sounds a bit weird
Codecov Report
@@ Coverage Diff @@
## master #2419 +/- ##
==========================================
+ Coverage 89.95% 90.13% +0.18%
==========================================
Files 213 213
Lines 34354 34328 -26
==========================================
+ Hits 30904 30943 +39
+ Misses 3450 3385 -65
Continue to review full report at Codecov.
|
Create more than 2 dimensions and verify that dimensions view displays information correctly.
Create more than 2 dimensions and verify
that dimensions view displays information correctly.