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.

Sign up for GitHub

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

Jump to bottom

Enable synthetic source by default if index.mode=time_series #92319

Closed
martijnvg opened this issue Dec 13, 2022 · 1 comment · Fixed by #93380
Closed

Enable synthetic source by default if index.mode=time_series #92319

martijnvg opened this issue Dec 13, 2022 · 1 comment · Fixed by #93380
Labels
:StorageEngine/TSDB You know, for Metrics Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)

Comments

@martijnvg
Copy link
Member

martijnvg commented Dec 13, 2022 •
edited
Loading

In the context of time series data streams synthetic source will be enabled most of the time, since _source is major disk space user. However currently synthetic source is enabled by default. We should change that if index.mode is set to time_series then synthetic source should be enabled. In the cases when the downsides of synthetic source matter, then synthetic source can still be disabled.
@martijnvg martijnvg added the :StorageEngine/TSDB You know, for Metrics label Dec 13, 2022
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo (Team:Analytics)

@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Dec 13, 2022
@martijnvg martijnvg mentioned this issue Dec 13, 2022
Closed
martijnvg added a commit to martijnvg/elasticsearch that referenced this issue Jan 31, 2023
@martijnvg
Enforce synthetic source for time series indices
6975601 
Closes elastic#92319
@martijnvg martijnvg mentioned this issue Jan 31, 2023
Merged
martijnvg added a commit that referenced this issue Feb 2, 2023
@martijnvg
Enforce synthetic source for time series indices (#93380)
9babcc9 
Support for synthetic source is also added to `unsigned_long` field as part of this change.
This is required because `unsigned_long` field types can be used in tsdb indices and
this change would prohibit the usage of these field type otherwise.

Closes #92319
mark-vieira pushed a commit to mark-vieira/elasticsearch that referenced this issue Feb 2, 2023
@martijnvg @mark-vieira
Enforce synthetic source for time series indices (elastic#93380)
0429fac 
Support for synthetic source is also added to `unsigned_long` field as part of this change.
This is required because `unsigned_long` field types can be used in tsdb indices and
this change would prohibit the usage of these field type otherwise.

Closes elastic#92319
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:StorageEngine/TSDB You know, for Metrics Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Enforce synthetic source for time series indices martijnvg/elasticsearch
2 participants
@martijnvg @elasticsearchmachine