Skip to content

Conversation

@nicolas-grekas
Copy link
Member

@nicolas-grekas nicolas-grekas commented Aug 4, 2016

Q A
Branch? master
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR symfony/symfony-docs#6858

This PR replaces TagAwareRedisAdapter introduced in #19047 by a generic TagAwareAdapter that works with any two adapters.

@nicolas-grekas
Copy link
Member Author

PR is now independent for any other one and can be reviewed (and merged) on its own.

* file that was distributed with this source code.
*/

namespace Symfony\Component\Cache\Adapter;
Copy link
Member Author

Choose a reason for hiding this comment

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

The changes on this file are just resetting it to its exact 3.1 state.

@nicolas-grekas nicolas-grekas force-pushed the cache-tag-aware branch 3 times, most recently from 90c0a03 to 1ccd974 Compare August 5, 2016 12:01
@nicolas-grekas nicolas-grekas merged commit 288308b into symfony:master Aug 7, 2016
nicolas-grekas added a commit that referenced this pull request Aug 7, 2016
…TagAwareRedisAdapter) (nicolas-grekas)

This PR was merged into the 3.2-dev branch.

Discussion
----------

[Cache] Add generic TagAwareAdapter wrapper (replaces TagAwareRedisAdapter)

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | symfony/symfony-docs#6858

This PR replaces TagAwareRedisAdapter introduced in #19047 by a generic `TagAwareAdapter` that works with any two adapters.

Commits
-------

288308b [Cache] Add generic TagAwareAdapter wrapper (replaces TagAwareRedisAdapter)
@nicolas-grekas nicolas-grekas deleted the cache-tag-aware branch August 7, 2016 15:41
@fabpot fabpot mentioned this pull request Oct 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants