Skip to content

Conversation

@nkaradzhov
Copy link

Update programmability examples to use the new Command interface
Add programmability link in the readme

+ add Programmability section to README and
@nkaradzhov
Copy link
Author

Hi @bobymicroby,

It turns out both examples wont run on master branch because of some missing exports:
defineScript ( which was present in [email protected] because main package did export * from '@redis/client'; and now this line is commented in this commit )
CommandParser , RedisArgument, NumberReply - all seem to be newly introduced and not exported to end user

@bobymicroby bobymicroby merged commit 53e3575 into bobymicroby:update-docs Apr 30, 2025
bobymicroby pushed a commit that referenced this pull request Oct 6, 2025
…redis#3077)

* refactor(test): improve test scenario reliability and maintainability

* tests: add resp3 check test (#1)

* test: refactor connection handoff tests with enhanced spy utility (redis#2)

* test: add comprehensive push notification disabled scenarios (redis#3)

* tests: add params config tests (redis#4)

* tests: add feature enablement tests (redis#5)

---------

Co-authored-by: Nikolay Karadzhov <[email protected]>
bobymicroby pushed a commit that referenced this pull request Nov 13, 2025
…s#3098)

* fix(ssubscribe): properly resubscribe in case of shard failover

1) when RE failover happens, there is a disconnect
2) affected Client reconnects and tries to resubscribe all existing listeners
ISSUE #1: CROSSSLOT Error - client was doing ssubscribe ch1 ch2.. chN which, after the failover could result in CROSSSLOT ( naturally, becasuse now some slots could be owned by other shards )
FIX: send one ssubscribe command per channel instead of one ssubscribe for all channels
ISSUE redis#2: MOVED Error - some/all of the channels might be moved somewhere else
FIX: 1: Propagate the error to the Cluster. 2: Cluster rediscovers topology.
3: Cluster resubscribes all listeners of the failed client ( possibly some/all of those will end up in a different client after the rediscovery ) 

fixes: redis#2902
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants