Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[Cache] Make sure the chain cache configuration works#11813

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
javiereguiluz merged 1 commit intosymfony:3.4fromNyholm:patch-8
Jun 25, 2019

Conversation

@Nyholm
Copy link
Member

@NyholmNyholm commentedJun 23, 2019
edited
Loading

I just tried the configuration and noticed a few issues.

Im not 100% sure this actually works. Lets wait forsymfony/symfony#32139

Im sure now

@Nyholm
Copy link
MemberAuthor

I've updated the PR.

I tested the new config and it works perfectly.

@javiereguiluz
Copy link
Member

@Nyholm can you please confirm that this must go to 4.3 branch and not to 3.4 or 4.2? Thanks.

@Nyholm
Copy link
MemberAuthor

Nyholm commentedJun 25, 2019
edited
Loading

Sorry, no. This is equally true for 3.4. All active versions of the docs should be updated with this PR.

@javiereguiluzjaviereguiluz added this to the3.4 milestoneJun 25, 2019
@javiereguiluz
Copy link
Member

Thanks Tobias.

@javiereguiluzjaviereguiluz merged commit9fbebaf intosymfony:3.4Jun 25, 2019
javiereguiluz added a commit that referenced this pull requestJun 25, 2019
…yholm)This PR was submitted for the 4.3 branch but it was squashed and merged into the 3.4 branch instead (closes#11813).Discussion----------[Cache] Make sure the chain cache configuration worksI just tried the configuration and noticed a few issues.~~Im not 100% sure this actually works.~~ Lets wait forsymfony/symfony#32139Im sure nowCommits-------9fbebaf [Cache] Make sure the chain cache configuration works
@NyholmNyholm deleted the patch-8 branchJune 25, 2019 08:24
@Nyholm
Copy link
MemberAuthor

Thank you for the review and thank you for merging

fabpot added a commit to symfony/symfony that referenced this pull requestJul 5, 2019
…y providing several adapters (nicolas-grekas)This PR was merged into the 4.4 branch.Discussion----------[FrameworkBundle] Allow creating chained cache pools by providing several adapters| Q             | A| ------------- | ---| Branch?       | 4.4| Bug fix?      | no| New feature?  | yes| BC breaks?    | no| Deprecations? | no| Tests pass?   | yes| Fixed tickets | -| License       | MIT| Doc PR        | -Replaces#30984, followssymfony/symfony-docs#11813This PR allows defining several adapters for one pool. When doing so, this defines a chain pool.The benefit is that all chained pools get automatic namespace and lifetime, so things are transparent:```yamlpools:    my_chained_pool:        default_lifetime: 12        adapters:          - cache.adapter.array          - cache.adapter.filesystem          - {name: cache.adapter.redis, provider: 'redis://foo'}```(see fixtures for example of PHP/XML config)/cc@Nyholm@pborreli FYICommits-------29ba091 [FrameworkBundle] Allow creating chained cache pools by providing several adapters
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@nicolas-grekasnicolas-grekasnicolas-grekas approved these changes

Assignees

No one assigned

Projects

None yet

Milestone

3.4

Development

Successfully merging this pull request may close these issues.

4 participants

@Nyholm@javiereguiluz@nicolas-grekas@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp