Skip to content

Conversation

@juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Nov 20, 2025

Local cache is not ideal as it is in most cases with APCu limited in since and on systems with lots of users and with load balanced webserver nodes it is not likely that this cache is hit.

Moving to a distributed one should make sure this is more likely to have a cache hit on big instances and overhead to the distributed should be ok with a page load request doing one cache check

Signed-off-by: Julius Knorr <jus@bitgrid.net>
@juliusknorr juliusknorr requested a review from artonge November 20, 2025 15:44
@juliusknorr juliusknorr added 3. to review Waiting for reviews performance 🚀 Performance issues and optimisations labels Nov 20, 2025
@artonge artonge merged commit e0c00d5 into master Nov 20, 2025
52 of 54 checks passed
@artonge artonge deleted the perf/distributed-cache branch November 20, 2025 16:34
@juliusknorr
Copy link
Member Author

/backport to stable32 please

@backportbot backportbot bot added the backport-request Pending backport by the backport-bot label Nov 20, 2025
@juliusknorr
Copy link
Member Author

/backport to stable31 please

@backportbot backportbot bot removed the backport-request Pending backport by the backport-bot label Nov 20, 2025
@backportbot
Copy link

backportbot bot commented Nov 20, 2025

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

# Switch to the target branch and update it
git checkout stable31
git pull origin stable31

# Create the new backport branch
git checkout -b backport/3328/stable31

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick 5c67c5a0

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/3328/stable31

Error: Failed to check for changes with origin/stable31: No changes found in backport branch


Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@juliusknorr
Copy link
Member Author

/backport to stable30 please

@backportbot backportbot bot added the backport-request Pending backport by the backport-bot label Nov 20, 2025
@backportbot
Copy link

backportbot bot commented Nov 20, 2025

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

# Switch to the target branch and update it
git checkout stable30
git pull origin stable30

# Create the new backport branch
git checkout -b backport/3328/stable30

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick 5c67c5a0

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/3328/stable30

Error: Failed to check for changes with origin/stable30: No changes found in backport branch


Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

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

Labels

3. to review Waiting for reviews backport-request Pending backport by the backport-bot performance 🚀 Performance issues and optimisations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants