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

improve search result loading times #187

Merged
merged 9 commits into from
Dec 7, 2021
Merged

improve search result loading times #187

merged 9 commits into from
Dec 7, 2021

Conversation

rkrenn
Copy link
Collaborator

@rkrenn rkrenn commented Nov 29, 2021

With the latest new feature (parent user hierarchy #177 ) particular DTO graphs tend to take a bit more time to compute. It manifests subtle eg. with the loading times of the (queryeditor) search result lists.

Profiling shows the rowsexpansion areas of eg. a row of trial search results shows increased time. It will be adressed by

  • load rowexpansion data only when there actaully are expanded rowexpansion areas
  • the default page size is reduced to show (load) overall less records initially

@rkrenn rkrenn changed the title reduce search result pagesize improve queryeditor search result loading times Nov 30, 2021
@rkrenn rkrenn changed the title improve queryeditor search result loading times improve search result loading times Nov 30, 2021
@rkrenn rkrenn linked an issue Dec 3, 2021 that may be closed by this pull request
@sonarcloud
Copy link

sonarcloud bot commented Dec 3, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@rkrenn rkrenn merged commit a3ce7be into master Dec 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

loading search result seems to be slower
1 participant