[GH-ISSUE #90] Huge Performance Issue #72

Closed
opened 2026-02-25 21:31:08 +03:00 by kerem · 3 comments
Owner

Originally created by @ciur on GitHub (Aug 20, 2020).
Original GitHub issue: https://github.com/ciur/papermerge/issues/90

Testing with ~ 200 files in main browser.
Main xhr request took 10s to return all files!
That is unacceptable!

Originally created by @ciur on GitHub (Aug 20, 2020). Original GitHub issue: https://github.com/ciur/papermerge/issues/90 Testing with ~ 200 files in main browser. Main xhr request took 10s to return all files! That is unacceptable!
kerem 2026-02-25 21:31:08 +03:00
Author
Owner

@ciur commented on GitHub (Aug 20, 2020):

This commit boosts browse request by 50%.
From 7s -> 3.5s.

Performance penalty comes from large number of request.user.has_perm(node, perm) calls .

<!-- gh-comment-id:677503150 --> @ciur commented on GitHub (Aug 20, 2020): [This commit](https://github.com/ciur/papermerge/commit/3257e7bbbeb680df4867614ef9087e7027c34113) boosts browse request by 50%. From 7s -> 3.5s. Performance penalty comes from large number of request.user.has_perm(node, perm) calls .
Author
Owner

@ciur commented on GitHub (Aug 20, 2020):

Yey! managed to lower GET /browse/ request time to ~ 1.9s. It is still lot of time for only 200 files.... 👎 still, is some progress...

Now I understand why redis is so popular !

<!-- gh-comment-id:677778833 --> @ciur commented on GitHub (Aug 20, 2020): Yey! [managed to lower](https://github.com/ciur/papermerge/commit/4eb519166e3fe4e3a2e7ba65399b1aada9b8235e) GET /browse/ request time to ~ 1.9s. It is still lot of time for only 200 files.... :-1: still, is some progress... Now I understand why redis is so popular !
Author
Owner

@ciur commented on GitHub (Aug 22, 2020):

Ok, lowered it down to 1s. For total of 200 files in directory, 1s request is OK. Not perfect, but acceptable.

<!-- gh-comment-id:678617818 --> @ciur commented on GitHub (Aug 22, 2020): Ok, lowered it down to 1s. For total of 200 files in directory, 1s request is OK. Not perfect, but acceptable.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/papermerge#72
No description provided.