-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[apm] use IndexPattern instead of IIndexPattern #107187
Conversation
@elasticmachine merge upstream |
Pinging @elastic/apm-ui (Team:apm) |
💚 Build SucceededMetrics [docs]History
To update your PR or re-run it, just comment with: |
const res = convertKueryToEsQuery(inputValue, indexPattern); | ||
const res = convertKueryToEsQuery( | ||
inputValue, | ||
indexPattern as IndexPattern |
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.
Is as IndexPattern
really needed?
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.
it is, indexPattern
is typed as IndexPatternTitleAndFields
which convertKueryToEsQuery
complains about.
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.
lgtm. just one nit
* use IndexPattern instead of IIndexPattern * typefix Co-authored-by: Kibana Machine <[email protected]>
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
* use IndexPattern instead of IIndexPattern * typefix Co-authored-by: Kibana Machine <[email protected]> Co-authored-by: Matthew Kime <[email protected]>
* use IndexPattern instead of IIndexPattern * typefix Co-authored-by: Kibana Machine <[email protected]>
Summary
Use
IndexPattern
instead ofIIndexPattern
asIIndexPattern
is deprecated due to too much ambiguity in its definition.