-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Sample data usage collector es client migration #86657
Sample data usage collector es client migration #86657
Conversation
Pinging @elastic/kibana-core-ui (Team:Core UI) |
@elastic/kibana-core With the team changes that have happened recently, it turns out that Core actually owns most of what the Core-UI team used to own. The changes haven't been reflected in the |
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.
I think the IDE auto-import got the type SearchResponse
from the wrong library. Once that is fixed, it LGTM.
@@ -17,6 +17,7 @@ | |||
* under the License. | |||
*/ | |||
|
|||
import { SearchResponse } from 'elasticsearch'; |
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.
I think this should be imported from the new client @elastic/elasticsearch
instead
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.
When I change the import location, my IDE errors with Module '"../../../../../../../node_modules/@elastic/elasticsearch"' has no exported member 'SearchResponse'.ts(2305)
Importing from src/core/server
works though.
💚 Build SucceededMetrics [docs]Distributable file count
History
To update your PR or re-run it, just comment with: |
Summary
Uses the new elasticsearch client in the
sample_data
usage collector.Part of #86358.
Checklist
Delete any items that are not applicable to this PR.
For maintainers