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

Fix enrich cache size setting name #117576

Closed
wants to merge 2 commits into from

Conversation

nielsbauman
Copy link
Contributor

The enrich cache size setting accidentally got renamed from enrich.cache_size to enrich.cache.size in #111412. This commit ensures we accept both versions, to avoid breaking BWC twice.

This is a manual (and altered) backport of #117575.

The enrich cache size setting accidentally got renamed from
`enrich.cache_size` to `enrich.cache.size` in elastic#111412. This commit
ensures we accept both versions, to avoid breaking BWC twice.
@nielsbauman nielsbauman added >bug :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP Team:Data Management Meta label for data/management team auto-backport Automatically create backport pull requests when merged v8.16.2 v8.17.1 v8.18.0 labels Nov 26, 2024
@elasticsearchmachine
Copy link
Collaborator

Hi @nielsbauman, I've created a changelog YAML for you.

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

@nielsbauman
Copy link
Contributor Author

Closing this one because #117575 now already incorporates these changes.

@nielsbauman nielsbauman closed this Dec 6, 2024
@nielsbauman nielsbauman deleted the enrich-fix-8.x branch December 6, 2024 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Automatically create backport pull requests when merged >bug :Data Management/Ingest Node Execution or management of Ingest Pipelines including GeoIP Team:Data Management Meta label for data/management team v8.16.2 v8.17.1 v8.18.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants