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

Script: no compile rate limit for ingest templates (#69841) #69987

Conversation

stu-elastic
Copy link
Contributor

Remove the compilation rate limit for ingest templates.

Creates a new context, ingest_template, with an unlimited
compilation rate limit.

The template context is used in many places so it cannot be
exempted from rate limits.

Fixes: #64595
Backport: 9370b1c

Remove the compilation rate limit for ingest templates.

Creates a new context, `ingest_template`, with an unlimited
compilation rate limit.

The `template` context is used in many places so it cannot be
exempted from rate limits.

Fixes: elastic#64595
Backport: 9370b1c
@stu-elastic stu-elastic added :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache backport v7.13.0 labels Mar 4, 2021
@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label Mar 4, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@stu-elastic stu-elastic merged commit 8e1a615 into elastic:7.x Mar 4, 2021
stu-elastic added a commit to stu-elastic/elasticsearch that referenced this pull request Mar 10, 2021
…lastic#69987)

Remove the compilation rate limit for ingest templates.

Creates a new context, `ingest_template`, with an unlimited
compilation rate limit.

The `template` context is used in many places so it cannot be
exempted from rate limits.

Fixes: elastic#64595
Backport: 9370b1c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache Team:Core/Infra Meta label for core/infra team v7.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants