Skip to content

pass the share to the cache instead of having to ask the storage - #39944

Merged
icewind1991 merged 1 commit into
masterfrom
share-cache-share
Aug 28, 2023
Merged

pass the share to the cache instead of having to ask the storage#39944
icewind1991 merged 1 commit into
masterfrom
share-cache-share

Conversation

@icewind1991

Copy link
Copy Markdown
Member

Going through the storage includes storage wrappers so it has a bit more overhead than you might expect.

A modest improvement for a small change

@icewind1991icewind1991 added this to the Nextcloud 28 milestone Aug 17, 2023
@icewind1991
icewind1991 requested review from a team, ArtificialOwl, blizzz and nfebe and removed request for a teamAugust 17, 2023 17:45
Signed-off-by: Robin Appelman <robin@icewind.nl>

@kesselbkesselb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice 👍

@icewind1991
icewind1991 merged commit d4e67af into masterAug 28, 2023
@icewind1991
icewind1991 deleted the share-cache-share branch August 28, 2023 11:44
@icewind1991

Copy link
Copy Markdown
MemberAuthor

/backport to stable27

@icewind1991

Copy link
Copy Markdown
MemberAuthor

/backport to stable26

@backportbot-nextcloud

Copy link
Copy Markdown

The backport to stable26 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable26
git pull origin stable26
# Create the new backport branch
git checkout -b fix/foo-stable26
# Cherry pick the change from the commit sha1 of the change against the default branch# This might cause conflicts. Resolve them.
git cherry-pick abc123
# Push the cherry pick commit to the remote repository and open a pull request
git push origin fix/foo-stable26

More info at https://docs.nextcloud.com/server/latest/developer_manual/getting_started/development_process.html#manual-backport

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsperformance 🚀

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants

@icewind1991@AndyScherzinger@kesselb@skjnldsv