FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Update xpending method to include idle-time parameter by dvigueras · Pull Request #2693 · phpredis/phpredis · GitHub

Update xpending method to include idle-time parameter - #2693

Open
dvigueras wants to merge 2 commits into
phpredis:developfrom
dvigueras:develop
Open

Update xpending method to include idle-time parameter#2693
dvigueras wants to merge 2 commits into
phpredis:developfrom
dvigueras:develop

Conversation

Copy link
Copy Markdown

This commit modifies the xpending method in both redis.stub.php and redis_cluster.stub.php to add an optional idle-time parameter. This allows users to filter pending messages based on idle time, enhancing the functionality and flexibility of the method.

This commit modifies the xpending method in both redis.stub.php
and redis_cluster.stub.php to add an optional idle-time parameter.
This allows users to filter pending messages based on idle time,
enhancing the functionality and flexibility of the method.

Copy link
Copy Markdown
Member

Thanks for the PR. We'll have to wait to merge this until putting together a major version bump since it changes the prototype which will break subclasses.

michael-grunder added the major-version-bump Things we can't merge until a major version change label Aug 25, 2025

michael-grunder commented Aug 25, 2025
edited
Loading

Copy link
Copy Markdown
Member

You'll need to regenerate the arginfo files with gen_stub.php

You can do this by cd'ing into the phpredis source tree and then running it like this:

php /usr/lib/php/20240924/build/gen_stub.php *.stub.php

It may be on a slightly different place on your system so you can search for it in /usr If it's not there php-config --prefix should show you where to check.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

major-version-bump Things we can't merge until a major version change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL