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
[Inspector] Make Response tab faster #180035
[Inspector] Make Response tab faster #180035
Conversation
/ci |
Pinging @elastic/kibana-data-discovery (Team:DataDiscovery) |
src/plugins/inspector/public/views/requests/components/details/req_code_viewer.tsx
Outdated
Show resolved
Hide resolved
src/plugins/inspector/public/views/requests/components/details/req_code_viewer.tsx
Outdated
Show resolved
Hide resolved
src/plugins/inspector/public/views/requests/components/details/req_code_viewer.tsx
Show resolved
Hide resolved
@nreese Thanks for reviewing! I refactored into using So here is a new tree structure:
|
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. Thanks for cleaning this up. That makes a lot of sense to not parse and process the response since its much larger then request and there is no need for all of this processing.
code review only
src/plugins/inspector/public/views/requests/components/details/req_details_request_content.tsx
Outdated
Show resolved
Hide resolved
…er' into make-inspector-response-tab-faster
💚 Build Succeeded
Metrics [docs]Module Count
Async chunks
History
To update your PR or re-run it, just comment with: cc @jughosta |
## Summary This PR improves the speed of "Response" tab rendering in the Inspector flyout. The issue was that the heavy logic of building links ("Open in Console", "Open in Search Profiler") was executed for Response tab too although links can appear only for Request tab. --------- Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com> (cherry picked from commit 7a00d2e)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
# Backport This will backport the following commits from `main` to `8.13`: - [[Inspector] Make Response tab faster (#180035)](#180035) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Julia Rechkunova","email":"julia.rechkunova@elastic.co"},"sourceCommit":{"committedDate":"2024-04-05T11:05:07Z","message":"[Inspector] Make Response tab faster (#180035)\n\n## Summary\r\n\r\nThis PR improves the speed of \"Response\" tab rendering in the Inspector\r\nflyout. The issue was that the heavy logic of building links (\"Open in\r\nConsole\", \"Open in Search Profiler\") was executed for Response tab too\r\nalthough links can appear only for Request tab.\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>","sha":"7a00d2e24b38bbdb5b41c31fe0658d78067ee679","branchLabelMapping":{"^v8.14.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","Feature:Inspector","Team:DataDiscovery","backport:prev-minor","v8.14.0"],"title":"[Inspector] Make Response tab faster","number":180035,"url":"#180035 Make Response tab faster (#180035)\n\n## Summary\r\n\r\nThis PR improves the speed of \"Response\" tab rendering in the Inspector\r\nflyout. The issue was that the heavy logic of building links (\"Open in\r\nConsole\", \"Open in Search Profiler\") was executed for Response tab too\r\nalthough links can appear only for Request tab.\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>","sha":"7a00d2e24b38bbdb5b41c31fe0658d78067ee679"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.14.0","branchLabelMappingKey":"^v8.14.0$","isSourceBranch":true,"state":"MERGED","url":"#180035 Make Response tab faster (#180035)\n\n## Summary\r\n\r\nThis PR improves the speed of \"Response\" tab rendering in the Inspector\r\nflyout. The issue was that the heavy logic of building links (\"Open in\r\nConsole\", \"Open in Search Profiler\") was executed for Response tab too\r\nalthough links can appear only for Request tab.\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>","sha":"7a00d2e24b38bbdb5b41c31fe0658d78067ee679"}}]}] BACKPORT--> Co-authored-by: Julia Rechkunova <julia.rechkunova@elastic.co>
Summary
This PR improves the speed of "Response" tab rendering in the Inspector flyout. The issue was that the heavy logic of building links ("Open in Console", "Open in Search Profiler") was executed for Response tab too although links can appear only for Request tab.