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 request chaining and associated overhead #3961

Open
ybnd opened this issue Feb 6, 2025 · 0 comments
Open

Reduce request chaining and associated overhead #3961

ybnd opened this issue Feb 6, 2025 · 0 comments
Assignees
Labels
bug performance / caching Related to performance, caching or embedded objects

Comments

@ybnd
Copy link
Member

ybnd commented Feb 6, 2025

As described in #3110 (comment) and #3110 (comment), many DSpace REST requests show multiple other requests as "initiators".

We're currently investigating the following avenues:

  • Reducing the amount of chained requests
    • The example in the comments above mentions search filters, which depend on a chain of at least the following requests:
      • /server/api → HAL endpoint map
      • /server/api/discover → HAL endpoint map for discovery
      • /server/api/discover/search → HAL endpoint map / default search configuration
      • /server/api/discover/objects and/or /server/api/discover/facets → determine which facets are visible
      • /server/api/discover/facets/<:id> → finally retrieve the actual filter
    • These can be direct (within the same RxJS pipe) or indirect (components rendering after a response is received and sending their own requests)
  • Assessing our total NgRx/RxJS overhead
    • We may want to try and tackle request chains by embedding more or pre-caching, but we'd still go through a long stack of NgRx selectors and RxJS operators
    • Intuitively we think that this shouldn't have too much impact, but it's not immediately clear how to measure this
@ybnd ybnd added the performance / caching Related to performance, caching or embedded objects label Feb 6, 2025
@ybnd ybnd self-assigned this Feb 6, 2025
@tdonohue tdonohue added the bug label Feb 6, 2025
@tdonohue tdonohue moved this from 📋 To Do to 🏗 In Progress in DSpace 8.x and 7.6.x Maintenance Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug performance / caching Related to performance, caching or embedded objects
Projects
Status: 🏗 In Progress
Development

No branches or pull requests

2 participants