Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Messenger] Added Lazy option to redis #14469

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
Merged
merged 1 commit into from
Oct 23, 2020
Merged

[Messenger] Added Lazy option to redis #14469

merged 1 commit into from
Oct 23, 2020

Conversation

Nyholm
Copy link
Member

@Nyholm Nyholm commented Oct 23, 2020

This will fix #14411

@Nyholm Nyholm force-pushed the issue-14411-redis-lazy branch from b75d73b to 9a30313 Compare October 23, 2020 13:23
@javiereguiluz javiereguiluz added this to the 5.2 milestone Oct 23, 2020
@Nyholm Nyholm force-pushed the issue-14411-redis-lazy branch from 9a30313 to 9b61abc Compare October 23, 2020 13:29
@javiereguiluz javiereguiluz force-pushed the issue-14411-redis-lazy branch from 9b61abc to 85b27e5 Compare October 23, 2020 13:31
@javiereguiluz
Copy link
Member

Thanks for this! While merging I reworded this a bit (because in the docs we never use this syntax -> "we ..."; it's always about the reader, so -> "you can ...", "your app ...", etc.). I also mentioned this option in the versionadded 5.2 directive.

@javiereguiluz javiereguiluz merged commit 6e0d5a6 into symfony:5.x Oct 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Messenger Status: Reviewed
Projects
None yet
Milestone
5.2
Development

Successfully merging this pull request may close these issues.

[Messenger][Redis] Adding support for lazy connect
3 participants
@Nyholm @javiereguiluz @carsonbot