Skip to content

[9.0] [ML] Anomaly Explorer: Prevent crash on anomaly table filter (#213075)#213499

Merged
kibanamachine merged 1 commit intoelastic:9.0from
kibanamachine:backport/9.0/pr-213075
Mar 7, 2025
Merged

[9.0] [ML] Anomaly Explorer: Prevent crash on anomaly table filter (#213075)#213499
kibanamachine merged 1 commit intoelastic:9.0from
kibanamachine:backport/9.0/pr-213075

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 9.0:

Questions ?

Please refer to the Backport tool documentation

…#213075)

Fix for: elastic#212569
From what I found, the issue was with the `useUrlStateService` after
changes introduced in elastic#203224,
which made the service more generic.

When filtering causes the `explorer` to remount the `AnomaliesTable`,
pagination state updates are triggered before the effect that sets
`setCallback.current` executes.

Initializing the ref with `setState` ensures its availability from the
first render.

https://github.com/user-attachments/assets/d1aa8409-56e5-4632-a5f2-82350b877db6
(cherry picked from commit 0210468)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Mar 6, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) March 6, 2025 23:25
@kibanamachine kibanamachine merged commit 2fd6274 into elastic:9.0 Mar 7, 2025
12 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #23 / aiops change point detection UI allows change point selection for detailed view

Metrics [docs]

Async chunks

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

id before after diff
aiops 452.0KB 452.0KB +1.0B
transform 475.8KB 475.8KB +1.0B
total +2.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
dataVisualizer 27.9KB 27.9KB +1.0B
ml 76.3KB 76.3KB +1.0B
total +2.0B

cc @rbrtj

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