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.

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

Modify Redis Backend #11141

Closed
wants to merge 172 commits into from
Closed

Modify Redis Backend #11141

wants to merge 172 commits into from

Conversation

pantaovay
Copy link

Changes

  1. Modify cache redis backend, delete _PHCR
  2. Cache redis backend supports "redis" client option

Why

  1. The redis cache backend use a special key _PHCR to store all keys used by cache. There is a issue: why the backend cache redis add the special key(statsKey) _PHCR #10905 .According to @Green-Cat ,it is disabled in 2.0.x branch which is not true. If use _PHCR, every time we set a cache, we have to run sAdd and set commands. Store all keys in _PHCR will cause it expands quickly and this will influence performance.
  2. The libmemecached backend has supplied the option to disable this feature.
  3. The cache backend doesn't support redis option which means that we can not pass a redis connection. So if we use modelsCache and modelsMetadata, there will be two redis connections. So I modify this part and let redis option be supported, and through the lifetime of a request we can use one redis connection.

@pantaovay
Copy link
Author

@sergeyklay Sorry for the inconvenience😢

andresgutierrez and others added 24 commits November 16, 2015 19:05
Added ability to change document class to be returned in ODM
if set value must be:
this->_readed = true;
phalcon#11042 - Fix setting relation should not always cause dirty state change
Memcache has an internal lifetime limit of 30 days, see http://stackoverflow.com/questions/1418324/memcache-maximum-key-expiration-time
Setting something upper to this limit leads to having blank writes with no error, breaking the purpose of the session.
min it is, sorry
Updated Oracle PDO Adapter to describe DATE column
rianorie and others added 28 commits February 17, 2016 01:39
Retrieving ondelete and onupdate key constraints in the MySQL adapter
Dispatcher does not set last handler
…dd unit tests to verify the default log level is applied across multiple logs.
[2.0.x] setLogLevel method to Multiple Logger
[2.0.x] Issue phalcon#11286 - Model property visibility and setter functionality
Fexed Gettext::prepareOptions visiblity

If currentColumn is missing use column parametr.

prevent:

ERROR: RuntimeException Object
(
    [message:protected] => Trying to call method getname on a non-object
...
)
Added a couple tests for \Phalcon\Http\Request
2. Cache redis backend supports "reds" client option
@pantaovay pantaovay closed this Feb 26, 2016
@pantaovay pantaovay deleted the 2.0.x branch February 26, 2016 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.