X Tutup
Skip to content

Update xpending method to include idle-time parameter#2693

Open
dvigueras wants to merge 2 commits intophpredis:developfrom
dvigueras:develop
Open

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

Conversation

@dvigueras
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.
@michael-grunder
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 michael-grunder added the major-version-bump Things we can't merge until a major version change label Aug 25, 2025
@michael-grunder
Copy link
Copy Markdown
Member

michael-grunder commented Aug 25, 2025

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.

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

X Tutup