forked from elastic/kibana
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[8.x] [Infra] Refactor useSearchSession to useReloadRequestTime for t…
…imestamp based refresh (elastic#205338) (elastic#205521) # Backport This will backport the following commits from `main` to `8.x`: - [[Infra] Refactor useSearchSession to useReloadRequestTime for timestamp based refresh (elastic#205338)](elastic#205338) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Gonçalo Rica Pais da Silva","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-03T16:44:52Z","message":"[Infra] Refactor useSearchSession to useReloadRequestTime for timestamp based refresh (elastic#205338)\n\n## Summary\r\n\r\nRefactoring `useSearchSession` to `useReloadRequestTime` to decouple the\r\nusage of search.session for refreshing, avoiding a re-rendering loop\r\nissue. This change is then propagated to various Lens charts to make use\r\nof that instead of relying on searchSessionId.\r\n\r\nCloses elastic#203412\r\n\r\n## How to test\r\n\r\n- Go to Infrastructure Inventory page in Observability\r\n- Enable Auto-Refresh\r\n- Click on a infra item (host/service/etc) to open the fly-out\r\n- Update the time range in the flyout\r\n- Things should update without spurious refreshes.\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine <[email protected]>","sha":"fe988fa55272a4ee6006a3a3ffc899a03fd4390c","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-minor","Team:obs-ux-infra_services"],"title":"[Infra] Refactor useSearchSession to useReloadRequestTime for timestamp based refresh","number":205338,"url":"https://github.com/elastic/kibana/pull/205338","mergeCommit":{"message":"[Infra] Refactor useSearchSession to useReloadRequestTime for timestamp based refresh (elastic#205338)\n\n## Summary\r\n\r\nRefactoring `useSearchSession` to `useReloadRequestTime` to decouple the\r\nusage of search.session for refreshing, avoiding a re-rendering loop\r\nissue. This change is then propagated to various Lens charts to make use\r\nof that instead of relying on searchSessionId.\r\n\r\nCloses elastic#203412\r\n\r\n## How to test\r\n\r\n- Go to Infrastructure Inventory page in Observability\r\n- Enable Auto-Refresh\r\n- Click on a infra item (host/service/etc) to open the fly-out\r\n- Update the time range in the flyout\r\n- Things should update without spurious refreshes.\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine <[email protected]>","sha":"fe988fa55272a4ee6006a3a3ffc899a03fd4390c"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/205338","number":205338,"mergeCommit":{"message":"[Infra] Refactor useSearchSession to useReloadRequestTime for timestamp based refresh (elastic#205338)\n\n## Summary\r\n\r\nRefactoring `useSearchSession` to `useReloadRequestTime` to decouple the\r\nusage of search.session for refreshing, avoiding a re-rendering loop\r\nissue. This change is then propagated to various Lens charts to make use\r\nof that instead of relying on searchSessionId.\r\n\r\nCloses elastic#203412\r\n\r\n## How to test\r\n\r\n- Go to Infrastructure Inventory page in Observability\r\n- Enable Auto-Refresh\r\n- Click on a infra item (host/service/etc) to open the fly-out\r\n- Update the time range in the flyout\r\n- Things should update without spurious refreshes.\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine <[email protected]>","sha":"fe988fa55272a4ee6006a3a3ffc899a03fd4390c"}}]}] BACKPORT--> Co-authored-by: Gonçalo Rica Pais da Silva <[email protected]>
- Loading branch information
1 parent
ced841e
commit 25ede9b
Showing
22 changed files
with
143 additions
and
147 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.