-
Notifications
You must be signed in to change notification settings - Fork 24.5k
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
Make time series start and end time settings required settings #98833
Labels
Comments
Pinging @elastic/es-analytics-geo (Team:Analytics) |
Pinging @elastic/es-storage-engine (Team:StorageEngine) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
The
index.time_series.start_time
andindex.time_series.end_time
settings are required for tsdb backing indices of a data stream. However for standalone tsdb indices these settings are optional with somewhat extreme default values,01-01--9999
and31-12-9999
respectively. I suspect this was done for ease of testing tsdb functionality in a standalone index.For tsdb data streams it shouldn't currently be possible to get into a situation that a backing index has no
index.time_series.start_time
orindex.time_series.end_time
settings. However in the past it was possible due a bug and I don't think it is watertight. I think we need also need to makeindex.time_series.start_time
andindex.time_series.end_time
settings required when defining tsdb indices. This would make the tsdb data stream validation easier and less error prone.Found as part of investigating #98792
The text was updated successfully, but these errors were encountered: