Skip to content
Snippets Groups Projects

update: don't serialize permissions on resources within container

Closed Calum Mackervoy requested to merge 299-optimize-permissions-serialization into master

Closes #299 (closed)

We think that Serializing permissions on every resource in a container is expensive. This MR changes the behaviour of LDPSerializer so that permissions are not serialized on child resources if they're in a container (/circles/, /circles/1/members/), but they are serialized if the child is called directly (/circles/1/)

@jbpasquier before we test this with the front-end, is there a staging that's already deployed with the performance test data (500 users, 500 projects), or if not is there one that I can use to measure the improvement please?

Edited by Benoit Alessandroni

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading