-
Notifications
You must be signed in to change notification settings - Fork 13
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
feat(queryExpressions): add buildBaseQueryExpression
to create the base LogQL expression
#311
Conversation
return `sum by (${field}) (count_over_time(${LOG_STREAM_SELECTOR_EXPR} | drop __error__ | ${field}!="" [$__auto]))`; | ||
return `sum by (${field}) (count_over_time(${buildBaseQueryExpression( | ||
sceneObject | ||
)} | drop __error__ | ${field}!="" [$__auto]))`; |
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.
do we need excess whitespace anymore?
Summarizing our slack thread: Sven mentioned we might be able to fix this with _variableDependency on the scene classes. |
thanks galen. I'll continue to look into that. |
ead7e4d
to
87ffc87
Compare
buildBaseQueryExpression
will create the basic LogQL expression instead of relying on AdhocFilters to do the trick. This allows to more easily adapt expressions where needed.The PR also renames the
VAR_FILTERS
toVAR_LABELS
to make it clear that that variable stores all indexed labels.Additional side effect: this fixes the "Go to Explore" button, that did not include
| logfmt
in some cases.Fixes #257