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

Reduce main bucket in summary visualizations #3812

Closed
Tracked by #4155
Desvelao opened this issue Jan 25, 2022 · 0 comments · Fixed by #4376
Closed
Tracked by #4155

Reduce main bucket in summary visualizations #3812

Desvelao opened this issue Jan 25, 2022 · 0 comments · Fixed by #4376
Assignees
Labels

Comments

@Desvelao
Copy link
Member

Desvelao commented Jan 25, 2022

Wazuh Elastic Rev Security
4.x 7.x 4xxx Basic, ODFE, Xpack
Browser
Chrome, Firefox, Safari, etc

Description
We need to enhance the performance of the summary visualization that has each dashboard. The visualization is not visible in the UI but is useful when generating the report on PDF. The objective is to improve the performance.

note: outdated
Tasks

  • Reduce the size property for the larger buckets of the aggregations.

Strategy

We decided to make the request and render this table in the backend of the app because of the following reasons:

  • The size bucket was increased in this PR to avoid truncating results, so reducing its size would bring back the truncated information problem.
  • Also evaluated that this table is only used when generating PDF reports, thus no dashboard needs to render it unless the user clicks "Generate Report".

Avoiding to load this table will improve the performance in most of the app dashboards.

@Desvelao Desvelao added the type/enhancement Enhancement issue label Jan 25, 2022
@asteriscos asteriscos self-assigned this Jul 18, 2022
@AlexRuiz7 AlexRuiz7 changed the title [Optimization] Reduce main bucket in summary visualizations Reduce main bucket in summary visualizations Jul 25, 2022
@gdiazlo gdiazlo moved this to Triage in Release 4.4.0 Aug 10, 2022
@gdiazlo gdiazlo moved this from Triage to In Progress in Release 4.4.0 Aug 10, 2022
@Desvelao Desvelao linked a pull request Aug 11, 2022 that will close this issue
@vikman90 vikman90 added this to the Release 4.4.0 milestone Aug 31, 2022
@asteriscos asteriscos moved this from In Progress to In Review in Release 4.4.0 Sep 6, 2022
Repository owner moved this from In Review to Done in Release 4.4.0 Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

5 participants