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
expose local and locking memcaches trough public interfaces #6571
Conversation
lib/public/ICacheFactory.php
Outdated
* create a distributed cache instance | ||
* | ||
* @param string $prefix | ||
* @return \OC\Memcache\Cache |
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.
Public interface please
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.
done
lib/public/ICacheFactory.php
Outdated
* create a local cache instance | ||
* | ||
* @param string $prefix | ||
* @return \OC\Memcache\Cache |
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.
Same here
c85e114
to
1183eb2
Compare
Codecov Report
@@ Coverage Diff @@
## master #6571 +/- ##
===========================================
- Coverage 53.06% 52.86% -0.2%
- Complexity 22552 22560 +8
===========================================
Files 1414 1413 -1
Lines 87741 88079 +338
Branches 1340 1340
===========================================
+ Hits 46561 46565 +4
- Misses 41180 41514 +334
|
@@ -31,7 +31,7 @@ | |||
*/ | |||
interface ICacheFactory{ | |||
/** | |||
* Get a memory cache instance | |||
* Get a distributed memory cache instance |
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.
Since we now have a dedicated method to get a distributed cache, would it make sense to deprecate this ambiguous method and remove it in the future?
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.
Yes!
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.
done
Signed-off-by: Robin Appelman <robin@icewind.nl>
1183eb2
to
388376c
Compare
Give apps access to the different caches.
Using the local instead of distributed cache where applicable can lower response times and improve scalability