-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
MLv2 metadata calculation validation #29956
Conversation
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.
Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @camsaul)
No failed tests 🎉 |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #29956 +/- ##
==========================================
+ Coverage 69.41% 69.45% +0.04%
==========================================
Files 2844 2840 -4
Lines 98504 98476 -28
Branches 12528 12507 -21
==========================================
+ Hits 68373 68395 +22
+ Misses 24774 24731 -43
+ Partials 5357 5350 -7
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 25 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
Follow-on to #29952: validate that we can calculate metadata for all pMBQL queries. (This doesn't validate that it is correct, just that it doesn't fail).
So far this has caught the following bugs:
:breakout
from joins #29907type-of
calculation doesn't work fordatetime-add
,datetime-subtract
, orconvert-timezone
with temporal string literals #29910type-of
not working for aggregations containing:case
expression #29935:metric
clause #29936:field <name>
clauses #29941:ends-with
is broken #29947:percentile
column name calculation broken #29964Prereq for #29017 and #29825
This change is