Skip to content

[Cache] New array adapter cache #9110

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
Closed
weaverryan opened this issue Jan 23, 2018 · 1 comment
Closed

[Cache] New array adapter cache #9110

weaverryan opened this issue Jan 23, 2018 · 1 comment
Labels
Cache hasPR A Pull Request has already been submitted for this issue.
Milestone
4.1

Comments

@weaverryan
Copy link
Member

See symfony/symfony#25892

Also, our reference.rst section about the different cache adapters isn't great :).

The cache adapter used by the cache.app service. The FrameworkBundle
ships with multiple adapters: apcu, doctrine, system, filesystem,
psr6, redis and memcached.

It's not clear what values you would actually use, like cache.adapter.redis. It's also not very clear how you would configure, for example, the memcached connection details.
@weaverryan weaverryan added the Waiting Code Merge Docs for features pending to be merged label Jan 23, 2018
@weaverryan weaverryan mentioned this issue Jan 23, 2018
Merged
symfony-splitter pushed a commit to symfony/framework-bundle that referenced this issue Jan 31, 2018
@nicolas-grekas
feature #25892 Adding an array adapter (weaverryan)
3382700 
This PR was merged into the 4.1-dev branch.

Discussion
----------

Adding an array adapter

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

There's no simple way to *disable* caching (e.g. in the dev) environment. Setting up an array adapter is a very simple way to allow this :).

Commits
-------

f92e03d adding an array adapter
nicolas-grekas added a commit to symfony/symfony that referenced this issue Jan 31, 2018
@nicolas-grekas
feature #25892 Adding an array adapter (weaverryan)
7f8e215 
This PR was merged into the 4.1-dev branch.

Discussion
----------

Adding an array adapter

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

There's no simple way to *disable* caching (e.g. in the dev) environment. Setting up an array adapter is a very simple way to allow this :).

Commits
-------

f92e03d adding an array adapter
@javiereguiluz javiereguiluz added Missing Documentation Cache and removed Waiting Code Merge Docs for features pending to be merged labels Feb 5, 2018
@javiereguiluz javiereguiluz added this to the 4.1 milestone Feb 5, 2018
@javiereguiluz javiereguiluz changed the title [WCM] New array adapter cache [Cache] New array adapter cache Feb 5, 2018
@javiereguiluz javiereguiluz added the hasPR A Pull Request has already been submitted for this issue. label Feb 5, 2018
This was referenced Feb 5, 2018
Merged
Merged
javiereguiluz added a commit that referenced this issue Feb 6, 2018
@javiereguiluz
minor #9202 Improve the cache config reference (javiereguiluz)
ec0865f 
This PR was merged into the 3.4 branch.

Discussion
----------

Improve the cache config reference

It fixes one of the problems mentioned in #9110.

Commits
-------

bb522f9 Improve the cache config reference
javiereguiluz added a commit that referenced this issue Feb 6, 2018
@javiereguiluz
minor #9203 Mentioned the new cache.adapter.array (javiereguiluz)
5bdcae7 
This PR was merged into the master branch.

Discussion
----------

Mentioned the new cache.adapter.array

This fixes the main problem described in #9110.

Commits
-------

982fdae Mentioned the new cache.adapter.array
@javiereguiluz
Copy link
Member

Fixed by #9202 and #9203.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Cache hasPR A Pull Request has already been submitted for this issue.
Projects
None yet
Milestone
4.1
Development

No branches or pull requests
2 participants
@javiereguiluz @weaverryan