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.

Sign up for GitHub

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

Jump to bottom

[alerting] add ignore_above to alerts params mappings to handle immense params #100726

Merged
merged 1 commit into from
May 27, 2021
Merged

[alerting] add ignore_above to alerts params mappings to handle immense params #100726

merged 1 commit into from
May 27, 2021

Commits on May 26, 2021

  1. [alerting] add ignore_above to alerts params mappings to handle immen… 

    …se params

resolves elastic#100607

This fixes a problem when very large parameters (over 32K bytes) are saved with
an alert.  Before this fix, an error from elasticsearch would be thrown with
the following message, and a 400 returned from create (and presumably update).

    Document contains at least one immense term in field=\"alert.params\"
    (whose UTF8 encoding is longer than the max length 32766), all of which
    were skipped.

After the fix, you can alerts with immense params can be saved and executed.

Note that the immense params will not be searchable, since they won't be indexed,
but that seems both unavoidable, and not a severe issue.
    @pmuellr
    pmuellr committed May 26, 2021
    Configuration menu
    Copy the full SHA
    aefbd39 View commit details
    Browse the repository at this point in the history