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.

Sign up for GitHub

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

Jump to bottom

ShardablePhpRedis backend #7

Open
wants to merge 16 commits into
base: 8.x-1.x
Choose a base branch
from
Open

ShardablePhpRedis backend #7

wants to merge 16 commits into from

Conversation

Berdir
Copy link
Member

@Berdir Berdir commented Jul 11, 2015

This builds on top of #5 but I merged it into the current implementation.

pounard and others added 15 commits April 18, 2015 15:57
@pounard
Added ShardablePhpRedis backend
d9fd625
@pounard
Fixed tests of ShardedPhpRedis backend
b617f22
@pounard
Added ShardedPhpRedis client implementation
81562c4
@pounard
System may arbitrary choose the wrong implementation for the cache ba…
508f53e 
…ckend tests
@pounard
Fixed potential invalid item load as valid when a tag key is being re…
3f2e76e 
…moved
@pounard
Removed stupidly added invalidator service in services.yml file
646309d
@pounard
Restored static cache into invalidator/checksum service
c33f9d1
@pounard
The weirdest bug ever, I must have done a typo somewhere...
0fe256e
@pounard
Added todolist
b925998
@pounard
RedisPrefixTrait::getNextIncrement() documentation
6d6f907
@pounard
Added a few notes into the TODOLIST
ab4171c
@pounard
Fixed a Drupal coding standard convention violation
38e2607
@Berdir
Merge the shardable backend
84d8285
@Berdir
Merge branch '8.x-1.x' into shardable
c65f905
@Berdir
Merge tests, use REQUEST_TIME, time() doesn't work in tests
1cfd62a
@Berdir Berdir mentioned this pull request Jul 13, 2015
Open
@pounard
Copy link
Contributor

pounard commented Jul 16, 2015

Nice, didn't see you made another pull request! Thanks.

blueprintmrk
blueprintmrk suggested changes Nov 11, 2016
View reviewed changes
Copy link

@blueprintmrk blueprintmrk left a comment

Choose a reason for hiding this comment

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

Add to pantheon repo

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

@blueprintmrk blueprintmrk blueprintmrk requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Berdir @pounard @blueprintmrk