Skip to content
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

[Search] Fix infinite loop on search index page #178244

Merged

Conversation

sphilipse
Copy link
Member

@sphilipse sphilipse commented Mar 7, 2024

Summary

This fixes a bug on the search index page where we were inadvertently clearing index data every time it was loaded.

@sphilipse sphilipse added release_note:skip Skip the PR/issue when compiling release notes Team:EnterpriseSearch v8.13.0 v8.14.0 labels Mar 7, 2024
@sphilipse sphilipse requested a review from a team March 7, 2024 16:25
Copy link
Member

@efegurkan efegurkan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the bug, thanks for the fix

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

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
enterpriseSearch 2.7MB 2.8MB +376.0B

History

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

@sphilipse sphilipse merged commit cc2c364 into elastic:main Mar 8, 2024
19 checks passed
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Mar 8, 2024
## Summary

This fixes a bug on the search index page where we were inadvertently
clearing index data every time it was loaded.

(cherry picked from commit cc2c364)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.13

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Mar 8, 2024
…8301)

# Backport

This will backport the following commits from `main` to `8.13`:
- [[Search] Fix infinite loop on search index page
(#178244)](#178244)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Sander
Philipse","email":"94373878+sphilipse@users.noreply.github.com"},"sourceCommit":{"committedDate":"2024-03-08T12:12:31Z","message":"[Search]
Fix infinite loop on search index page (#178244)\n\n##
Summary\r\n\r\nThis fixes a bug on the search index page where we were
inadvertently\r\nclearing index data every time it was
loaded.","sha":"cc2c364d6061d514415d213b9a7540345e88e13c","branchLabelMapping":{"^v8.14.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:EnterpriseSearch","v8.13.0","v8.14.0"],"title":"[Search]
Fix infinite loop on search index
page","number":178244,"url":"https://github.com/elastic/kibana/pull/178244","mergeCommit":{"message":"[Search]
Fix infinite loop on search index page (#178244)\n\n##
Summary\r\n\r\nThis fixes a bug on the search index page where we were
inadvertently\r\nclearing index data every time it was
loaded.","sha":"cc2c364d6061d514415d213b9a7540345e88e13c"}},"sourceBranch":"main","suggestedTargetBranches":["8.13"],"targetPullRequestStates":[{"branch":"8.13","label":"v8.13.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.14.0","branchLabelMappingKey":"^v8.14.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/178244","number":178244,"mergeCommit":{"message":"[Search]
Fix infinite loop on search index page (#178244)\n\n##
Summary\r\n\r\nThis fixes a bug on the search index page where we were
inadvertently\r\nclearing index data every time it was
loaded.","sha":"cc2c364d6061d514415d213b9a7540345e88e13c"}}]}]
BACKPORT-->

Co-authored-by: Sander Philipse <94373878+sphilipse@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes Team:EnterpriseSearch v8.13.0 v8.14.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants