Skip to content

[8.18] [Security Solution] Fixes related integrations render performance on rule editing pages (#217254)#218669

Merged
kibanamachine merged 1 commit intoelastic:8.18from
kibanamachine:backport/8.18/pr-217254
Apr 18, 2025
Merged

[8.18] [Security Solution] Fixes related integrations render performance on rule editing pages (#217254)#218669
kibanamachine merged 1 commit intoelastic:8.18from
kibanamachine:backport/8.18/pr-217254

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.18:

Questions ?

Please refer to the Backport tool documentation

…rule editing pages (elastic#217254)

## Summary

Fixes elastic#183607

Adds logic to fix the re-render performance issues caused by the related
integrations component on the rule edit and creation pages. This copies
a strategy used in elastic#180682 to fix
a similar issue with required fields. Related integrations component now
doesn't re-render when there are updates to components that don't affect
it.

#### React Profile while typing in query field component
![Screenshot 2025-04-04 at 8 12
38 PM](https://github.com/user-attachments/assets/9d3edcaa-4856-42df-9e6d-59bcc4785b5d)

### Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
(cherry picked from commit 1b6376e)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Apr 18, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) April 18, 2025 18:53
@kibanamachine kibanamachine merged commit 1af5b25 into elastic:8.18 Apr 18, 2025
11 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
securitySolution 7030 7031 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 9.1MB 9.1MB +223.0B

cc @dplumlee

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.

3 participants