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

Perfomance problems in path-viewer #814

Open
vrozaev opened this issue Oct 16, 2024 · 1 comment
Open

Perfomance problems in path-viewer #814

vrozaev opened this issue Oct 16, 2024 · 1 comment
Assignees

Comments

@vrozaev
Copy link
Collaborator

vrozaev commented Oct 16, 2024

Problem

Right now path-viewer has perfomance problems, which a especially clear on default case.

By default, path-viewer trying to make get / command, which leads to massive response. Interface trying to render whole response in the same time and it's leads to interface lags and might even doesn't allow to use path-viewer.

Solution

There is no clear solution at the moment, but next things could be done on ui side:

  1. Process loaded data: BFS from the root and stop when you have enough data (make some limit). Mark everything else as truncated.
  2. Introduce virtualised renderer for the data (render only chunk of data which a fit at the browser screen).
  3. Introduce pagination for response.
  4. Replace default get / command to list - at least this change prevent perfomance problems right after path-viewer open.
@ma-efremoff
Copy link
Collaborator

ma-efremoff commented Oct 16, 2024

1,3 require corresponding API-features. As I know at the moment API does not provide them.

2. looks like a good idea, but we have to keep in mind any virtualized component may struggle with large data.
4. Use list by default looks like a cheap and safe change.

@vrozaev vrozaev self-assigned this Oct 18, 2024
vrozaev added a commit that referenced this issue Oct 18, 2024
… "get /" command might lead to perfomance issue [#814]
ma-efremoff pushed a commit that referenced this issue Oct 18, 2024
… "get /" command might lead to perfomance issue [#814]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants