-
-
Couldn't load subscription status.
- Fork 4.6k
[stable31] fix: more optimized caching for share target verification #52071
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.
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
base: stable31
Are you sure you want to change the base?
Conversation
Signed-off-by: Robin Appelman <robin@icewind.nl>
2d40b6d to
d125e81
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Has errors and reduces cache hits drastically
PR breaks prod, so invalidating any review
Signed-off-by: Robin Appelman <robin@icewind.nl>
|
added #52128 which should un-break things |
|
CI is now all greens, so merge or still blocked? |
|
Should test it on our instance again before? |
Yes, that be best ( if we can do that 👍 ) |
What to do with this PR @icewind1991 @nickvergessen |
|
This patch significantly worsens the performance on our instance causing a much higher load (2-5x times) |
Backport of #52045
Warning, This backport's changes differ from the original and might be incomplete⚠️ Todo
Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.