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
[ML] Data Frame Analytics maps view: ensure nodes reload correctly after using timepicker refresh #145265
[ML] Data Frame Analytics maps view: ensure nodes reload correctly after using timepicker refresh #145265
Conversation
Pinging @elastic/ml-ui (:ml) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and LGTM
@elasticmachine merge upstream |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - was able to reproduce the issue on main
and the fix in this PR.
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]Async chunks
Unknown metric groupsESLint disabled in files
ESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: |
…ter using timepicker refresh (elastic#145265) ## Summary Fixes elastic#144308 This PR adds tracking of node ids that have been used to fetch additional node data so that, on refresh, the data for these nodes is also refetched. Before: ![dfa_maps_crash](https://user-images.githubusercontent.com/1945390/199214556-2b165ac5-26d6-4e45-bfe0-a531494df40f.gif) After: https://user-images.githubusercontent.com/6446462/201964406-39689a3c-76f9-4132-ad62-5c2bccc17622.mp4 ### Checklist Delete any items that are not applicable to this PR. - [x] Any UI touched in this PR is usable by keyboard only (learn more about [keyboard accessibility](https://webaim.org/techniques/keyboard/)) Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> (cherry picked from commit baf450d)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…tly after using timepicker refresh (#145265) (#145607) # Backport This will backport the following commits from `main` to `8.6`: - [[ML] Data Frame Analytics maps view: ensure nodes reload correctly after using timepicker refresh (#145265)](#145265) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Melissa Alvarez","email":"melissa.alvarez@elastic.co"},"sourceCommit":{"committedDate":"2022-11-17T18:32:02Z","message":"[ML] Data Frame Analytics maps view: ensure nodes reload correctly after using timepicker refresh (#145265)\n\n## Summary\r\n\r\nFixes #144308 PR adds tracking of node ids that have been used to fetch\r\nadditional node data so that, on refresh, the data for these nodes is\r\nalso refetched.\r\n\r\nBefore:\r\n\r\n\r\n![dfa_maps_crash](https://user-images.githubusercontent.com/1945390/199214556-2b165ac5-26d6-4e45-bfe0-a531494df40f.gif)\r\n\r\nAfter:\r\n\r\n\r\nhttps://user-images.githubusercontent.com/6446462/201964406-39689a3c-76f9-4132-ad62-5c2bccc17622.mp4\r\n\r\n\r\n### Checklist\r\n\r\nDelete any items that are not applicable to this PR.\r\n- [x] Any UI touched in this PR is usable by keyboard only (learn more\r\nabout [keyboard accessibility](https://webaim.org/techniques/keyboard/))\r\n\r\nCo-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>","sha":"baf450ddde6726b9523e20519585caa002cd8197","branchLabelMapping":{"^v8.7.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix",":ml","Feature:Data Frame Analytics","v8.6.0","v8.7.0"],"number":145265,"url":"#145265 Data Frame Analytics maps view: ensure nodes reload correctly after using timepicker refresh (#145265)\n\n## Summary\r\n\r\nFixes #144308 PR adds tracking of node ids that have been used to fetch\r\nadditional node data so that, on refresh, the data for these nodes is\r\nalso refetched.\r\n\r\nBefore:\r\n\r\n\r\n![dfa_maps_crash](https://user-images.githubusercontent.com/1945390/199214556-2b165ac5-26d6-4e45-bfe0-a531494df40f.gif)\r\n\r\nAfter:\r\n\r\n\r\nhttps://user-images.githubusercontent.com/6446462/201964406-39689a3c-76f9-4132-ad62-5c2bccc17622.mp4\r\n\r\n\r\n### Checklist\r\n\r\nDelete any items that are not applicable to this PR.\r\n- [x] Any UI touched in this PR is usable by keyboard only (learn more\r\nabout [keyboard accessibility](https://webaim.org/techniques/keyboard/))\r\n\r\nCo-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>","sha":"baf450ddde6726b9523e20519585caa002cd8197"}},"sourceBranch":"main","suggestedTargetBranches":["8.6"],"targetPullRequestStates":[{"branch":"8.6","label":"v8.6.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.7.0","labelRegex":"^v8.7.0$","isSourceBranch":true,"state":"MERGED","url":"#145265 Data Frame Analytics maps view: ensure nodes reload correctly after using timepicker refresh (#145265)\n\n## Summary\r\n\r\nFixes #144308 PR adds tracking of node ids that have been used to fetch\r\nadditional node data so that, on refresh, the data for these nodes is\r\nalso refetched.\r\n\r\nBefore:\r\n\r\n\r\n![dfa_maps_crash](https://user-images.githubusercontent.com/1945390/199214556-2b165ac5-26d6-4e45-bfe0-a531494df40f.gif)\r\n\r\nAfter:\r\n\r\n\r\nhttps://user-images.githubusercontent.com/6446462/201964406-39689a3c-76f9-4132-ad62-5c2bccc17622.mp4\r\n\r\n\r\n### Checklist\r\n\r\nDelete any items that are not applicable to this PR.\r\n- [x] Any UI touched in this PR is usable by keyboard only (learn more\r\nabout [keyboard accessibility](https://webaim.org/techniques/keyboard/))\r\n\r\nCo-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>","sha":"baf450ddde6726b9523e20519585caa002cd8197"}}]}] BACKPORT--> Co-authored-by: Melissa Alvarez <melissa.alvarez@elastic.co>
Summary
Fixes #144308
This PR adds tracking of node ids that have been used to fetch additional node data so that, on refresh, the data for these nodes is also refetched.
Before:
After:
dfaMapFix.mp4
Checklist
Delete any items that are not applicable to this PR.