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

Use non-metric fields for routing #103463

Closed
wants to merge 27 commits into from

Conversation

kkrik-es
Copy link
Contributor

Track all fields in dynamic template spec during bulk ingestion as metrics and assume that the rest are dimensions. These get then used for routing and _tsid calculations.

This mode is controlled with flag time_series_dynamic_templates and it's incompatible with setting index.routing_path, either explicitly in the template or implicitly through fields marked as 'time_series_dimension'.

Related to #98384

kkrik-es and others added 27 commits December 5, 2023 14:40
Dynamic templates allow defining fields with `time_series_dimension`
annotations. When these fields are referenced in dynamic template specs
at indexing time, they can be used for routing instead of the routing
path; the latter can be empty in case all dimensions are dynamically
specified.

Related to elastic#98384
# Conflicts:
#	server/src/main/java/org/elasticsearch/search/DefaultSearchContext.java
…ting' into prototype/dynamic_dimensions_setting
…ting' into prototype/dynamic_dimensions_setting
…ting' into prototype/dynamic_dimensions_setting
@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v8.13.0 labels Dec 14, 2023
@kkrik-es kkrik-es removed the needs:triage Requires assignment of a team area label label Dec 14, 2023
@kkrik-es kkrik-es added :StorageEngine/TSDB You know, for Metrics and removed v8.13.0 labels Dec 14, 2023
@kkrik-es kkrik-es closed this Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:StorageEngine/TSDB You know, for Metrics
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants