Skip to content

Conversation

@afharo
Copy link
Member

@afharo afharo commented Aug 19, 2020

Summary

Changes needed to be able to read the dataset and type information from the new data_stream fields in the mappings.

Resolves #74240

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@afharo afharo marked this pull request as ready for review August 19, 2020 12:52
@afharo afharo requested a review from a team as a code owner August 19, 2020 12:52
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-telemetry (Team:KibanaTelemetry)

@afharo
Copy link
Member Author

afharo commented Aug 21, 2020

@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
Copy link
Member Author

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 🙂

@afharo
Copy link
Member Author

afharo commented Aug 24, 2020

@elasticmachine merge upstream

1 similar comment
@afharo
Copy link
Member Author

afharo commented Aug 24, 2020

@elasticmachine merge upstream

@Bamieh
Copy link
Member

Bamieh commented Aug 24, 2020

@elasticmachine merge upstream

@afharo
Copy link
Member Author

afharo commented Aug 24, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@afharo afharo merged commit 8fe62c3 into elastic:master Aug 24, 2020
@afharo afharo deleted the data-telemetry/rename-dataset-to-data_stream branch August 24, 2020 16:32
afharo added a commit to afharo/kibana that referenced this pull request Aug 24, 2020
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
afharo added a commit that referenced this pull request Aug 24, 2020
…5789)

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@lukeelmers lukeelmers added the Team:Core Platform Core services: plugins, logging, config, saved objects, http, ES client, i18n, etc t// label Oct 1, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core (Team:Core)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Telemetry release_note:skip Skip the PR/issue when compiling release notes Team:Core Platform Core services: plugins, logging, config, saved objects, http, ES client, i18n, etc t// v7.10.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Telemetry] Data Telemetry: Rename dataset.* to data_stream.*

5 participants