Skip to content

[7.x] [Alerting] Improves performance of the authorization filter in AlertsClient.find by skipping KQL parsing (#77040)#77439

Merged
gmmorris merged 1 commit intoelastic:7.xfrom
gmmorris:backport/7.x/pr-77040
Sep 15, 2020
Merged

[7.x] [Alerting] Improves performance of the authorization filter in AlertsClient.find by skipping KQL parsing (#77040)#77439
gmmorris merged 1 commit intoelastic:7.xfrom
gmmorris:backport/7.x/pr-77040

Conversation

@gmmorris
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

…Client.find by skipping KQL parsing (elastic#77040)

The Alerts Authorisation now manually builds the authorization filter instead of parsing a KQL string in order to get around the performance issue we recently identified in KQL.
@gmmorris gmmorris added the backport This PR is a backport of another PR label Sep 15, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Build metrics

distributable file count

id value diff baseline
default 47224 +2 47222

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@gmmorris gmmorris merged commit 18d48aa into elastic:7.x Sep 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants