Skip to content
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

Add support for dimension fields #215

Closed
jsoriano opened this issue Aug 30, 2021 · 2 comments · Fixed by #236
Closed

Add support for dimension fields #215

jsoriano opened this issue Aug 30, 2021 · 2 comments · Fixed by #236
Assignees
Labels
7.16-candidate discuss Issue needs discussion Team:Integrations Label for the Integrations team

Comments

@jsoriano
Copy link
Member

jsoriano commented Aug 30, 2021

Dimensions in time series databases are used to identify different time series. Metrics in documents with the same set of dimensions are considered the same time serie. Metrics with different set of dimensions are different time series.
This is specially relevant relevant when querying time series.

There is an effort in Elasticsearch to include dimensions as first-class citizens (see elastic/elasticsearch#74014, elastic/elasticsearch#74450), what will have advantages when storing and consuming time series data. To leverage these features in integrations, their dimensions should be properly defined.

For that we could add a new boolean property "dimension". Fields with this property set to true will be considered dimensions, any other field will be considered fields, or tags in the case of other metadata.

Not all types of metrics can be dimensions (to confirm what types can be dimensions).

@jsoriano jsoriano added discuss Issue needs discussion Team:Integrations Label for the Integrations team labels Aug 30, 2021
@exekias
Copy link
Contributor

exekias commented Aug 30, 2021

++ on this, and the proposed dimension property under fields.yml.

If I understand latest changes, this would translate into a time_series_dimension mapping parameter. (BTW we may also need to update metric_type mapping once things get in).

Do you know if this already scheduled for a release on the ES side?

@jsoriano
Copy link
Member Author

Do you know if this already scheduled for a release on the ES side?

I think it is targeted for 7.16, see elastic/elasticsearch#74450

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
7.16-candidate discuss Issue needs discussion Team:Integrations Label for the Integrations team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants