Skip to content
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

memcached.store_retry_count default is incorrect in docs #2727

Closed
tabacco opened this issue Aug 31, 2023 · 0 comments · Fixed by #2728
Closed

memcached.store_retry_count default is incorrect in docs #2727

tabacco opened this issue Aug 31, 2023 · 0 comments · Fixed by #2728

Comments

@tabacco
Copy link
Contributor

tabacco commented Aug 31, 2023

With release 3.2.0 Memcached changed the default for store_retry_count from 2 to 0.

Changelog noting the change: https://pecl.php.net/package-changelog.php?package=memcached&release=3.2.0
PR for the change: https://github.com/php-memcached-dev/php-memcached/pull/452/files

The docs for the extension should be updated with the new default value.
tabacco added a commit to tabacco/doc-en that referenced this issue Aug 31, 2023
@tabacco
Update the default value for memcached.store_retry_count
85ce2de 
The default was changed in 3.2.0 - Fixes php#2727
@tabacco tabacco mentioned this issue Aug 31, 2023
Merged
Girgias pushed a commit that referenced this issue Sep 2, 2023
@tabacco
Update the default value for memcached.store_retry_count (#2728)
dfe3b95 
The default was changed in 3.2.0 - Fixes #2727
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update the default value for memcached.store_retry_count tabacco/doc-en
1 participant
@tabacco