-
Notifications
You must be signed in to change notification settings - Fork 11.8k
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
Elasticsearch: Improve query type selection #63402
Conversation
Backend code coverage report for PR #63402 |
Frontend code coverage report for PR #63402
|
7f2dd3a
to
3dbe646
Compare
This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 2 weeks if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions! |
keep it open please |
3dbe646
to
c8860c5
Compare
You have successfully added a new CodeQL configuration |
f1deb97
to
53b9d0a
Compare
096e43f
to
866ddec
Compare
866ddec
to
b52b498
Compare
( fixes #62216 )
in the elasticsearch query editor, currently, when you want to run a logs-query or a raw-data-query or a raw-document-query, you choose it by finding the right option in the "metric" select-box. this is hard to discover.
this pull request moves the 4 query types (metric, logs, raw-data, raw-doc) to the top of the query editor. and makes sure the logs, raw-data and raw-doc options do not show up in the metric-list anymore.
please note, this is purely a visual change, the underlying
ElasticQuery
data model does not change.additional changes:
query
tolucene query
, and adjusted the text-field placeholder-textfuture work: to decide which metric-options to hide we use the already existing
isSingleMetric
boolean. it already exists, because the reducers needed to know that when one of these (logs, raw-data, raw-doc) is chosen, all existing bucket-aggregations should be removed in the user-interface. i think we should rename it to something more descriptive, perhapsisSpecialMetric
(naming ideas very welcome 👍 )... but did not want to make the PR more complex, so will do in a separate PR later.how to test: