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 dynamic: false instead of dynamic: runtime for data streams #12097

Closed
carsonip opened this issue Nov 22, 2023 · 1 comment
Closed

Use dynamic: false instead of dynamic: runtime for data streams #12097

carsonip opened this issue Nov 22, 2023 · 1 comment
Assignees

Comments

@carsonip
Copy link
Member

carsonip commented Nov 22, 2023

In #10968 we set dynamic: runtime on internal metrics, all aggregated metrics, and TBS metrics. Out of all, internal metrics and all aggregated metrics are actually using synthetic source, meaning that the extra fields will not be indexed anyway. There is no value in creating a runtime field for a field that is not indexed. Consider moving to dynamic: false, as well as consider the case of TBS metrics because it does not use synthetic source.

Using dynamic runtime has caused issues for upgrades to 8.11.0 and 8.11.1: #12103

@axw
Copy link
Member

axw commented May 24, 2024

As of 8.15 we will no longer use dynamic: runtime -- we're not setting this in the apm-data plugin.

@axw axw closed this as completed May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants