Skip to content

Documented redis transport new stream_max_entries option #11870

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 3, 2019

Conversation

Toflar
Copy link
Contributor

@Toflar Toflar commented Jul 3, 2019

@javiereguiluz javiereguiluz changed the base branch from master to 4.4 July 3, 2019 07:51
@javiereguiluz javiereguiluz force-pushed the messenger-redis-maxlen branch from a5eb9ed to d1aa071 Compare July 3, 2019 07:51
javiereguiluz added a commit to javiereguiluz/symfony-docs that referenced this pull request Jul 3, 2019
… option (Toflar)

This PR was submitted for the master branch but it was merged into the 4.4 branch instead (closes symfony#11870).

Discussion
----------

Documented redis transport new stream_max_entries option

Docs for symfony/symfony#31825

Commits
-------

d1aa071 Documented redis transport new stream_max_entries option
@javiereguiluz javiereguiluz merged commit d1aa071 into symfony:4.4 Jul 3, 2019
@javiereguiluz
Copy link
Member

Thanks Yanick for this contribution! We merged it in the 4.4 branch and then, in master too.

@Toflar Toflar deleted the messenger-redis-maxlen branch July 3, 2019 07:58
@Toflar
Copy link
Contributor Author

Toflar commented Jul 3, 2019

Ah, wrong branch again 🙈 Thanks!

OskarStark added a commit that referenced this pull request Aug 19, 2019
…le (alexander-schranz)

This PR was merged into the 4.4 branch.

Discussion
----------

Add stream_max_entries to full redis messenger dsn example

As introduced in: #11870
Should also added to the full dsn example which was added in: #11874

Commits
-------

467d6c2 Add stream_max_entries to full redis messenger dsn example
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.

3 participants