-
Notifications
You must be signed in to change notification settings - Fork 8.5k
[Data Telemetry] Rename dataset.* to data_stream.* #75415
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
[Data Telemetry] Rename dataset.* to data_stream.* #75415
Conversation
|
Pinging @elastic/kibana-telemetry (Team:KibanaTelemetry) |
|
@elasticmachine merge upstream |
| export interface DataTelemetryDocument extends DataTelemetryBasePayload { | ||
| dataset?: { | ||
| name?: string; | ||
| type?: DataTelemetryType | 'unknown' | string; // The union of types is to help autocompletion with some known `dataset.type`s |
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.
'unknown' was there from the initial implementation. But now, if unknown, we simply don't report it 🙂
|
@elasticmachine merge upstream |
1 similar comment
|
@elasticmachine merge upstream |
|
@elasticmachine merge upstream |
|
@elasticmachine merge upstream |
💚 Build SucceededBuild metrics
History
To update your PR or re-run it, just comment with: |
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
|
Pinging @elastic/kibana-core (Team:Core) |
Summary
Changes needed to be able to read the
datasetandtypeinformation from the newdata_streamfields in the mappings.Resolves #74240
Checklist
Delete any items that are not applicable to this PR.
For maintainers