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.

Sign up for GitHub

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

Jump to bottom

Combined performance backport test #40274

Closed
wants to merge 11 commits into from
Closed

Combined performance backport test #40274

wants to merge 11 commits into from

Conversation

icewind1991
Copy link
Member

@icewind1991
more share permission logic to storage wrapper
ef2dd6b 
this way we only have to determine the share permissions once

Signed-off-by: Robin Appelman <[email protected]>
@icewind1991
cleanup di for share permissions wrapper
83e0458 
Signed-off-by: Robin Appelman <[email protected]>
@icewind1991
reuse l10n and request in dav folder listing
b5305e9 
instead of having to query those once for every node

Signed-off-by: Robin Appelman <[email protected]>
@icewind1991
don't bother checking dav acl's in files
9e07df3 
any permissions for files is handled in the filesystem layer anyway

Signed-off-by: Robin Appelman <[email protected]>
@icewind1991
reuse dependencies from wrapped cache
0201731 
Signed-off-by: Robin Appelman <[email protected]>
@icewind1991
only gather dependencies for trashbin wrapper once
09debc4 
Signed-off-by: Robin Appelman <[email protected]>
@icewind1991
better caching in storage stats calculations
80028ef 
Signed-off-by: Robin Appelman <[email protected]>
@icewind1991
only get logger when needed in sabre/directory
3e64be8 
Signed-off-by: Robin Appelman <[email protected]>
@icewind1991
only determine quota_include_external_storage once for quota wrapper
c255945 
Signed-off-by: Robin Appelman <[email protected]>
@icewind1991
pass the share to the cache instead of having to ask the storage
828ebd9 
Signed-off-by: Robin Appelman <[email protected]>
@icewind1991
improve performance of calculating dav permissions
54ead46 
Signed-off-by: Robin Appelman <[email protected]>
@jospoortvliet
Copy link
Member

so, how much faster is it ;-)

@solracsf solracsf added this to the Nextcloud 28 milestone Nov 21, 2023
@blizzz blizzz mentioned this pull request Nov 22, 2023
Merged
5 tasks
@blizzz blizzz modified the milestones: Nextcloud 28, Nextcloud 29 Nov 23, 2023
@skjnldsv skjnldsv added the 2. developing Work in progress label Feb 21, 2024
@icewind1991 icewind1991 closed this Mar 5, 2024
@skjnldsv skjnldsv deleted the combined-perf-26 branch March 14, 2024 07:51
@skjnldsv skjnldsv removed this from the Nextcloud 29 milestone Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
2. developing Work in progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@icewind1991 @jospoortvliet @blizzz @solracsf @skjnldsv