Skip to content

DNM: Reproduce RunStorage.read_many crash on redis-py RedisCluster - #843

Closed
chromy wants to merge 1 commit into
mainfrom
chromy/2026-11-04-redis-cluster-mget
Closed

chromy wants to merge 1 commit into
mainfrom
chromy/2026-11-04-redis-cluster-mget

Conversation

@chromy

@chromy chromy commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

No description provided.

…luster

On redis>=4.1, RedisCluster.mget raises unless every key shares a hash slot;
redis-py-cluster split the call per node. RunStorage.read_many mgets one key per
schedule, so the scheduler crashes on startup with more than one schedule.

Run the client tests on redis-py 5 (the cluster extra now conflicts with the dev
group), add a test reproducing the crash, and one showing that routing mget
through mget_nonatomic fixes it. Type-only changes in storage.py are needed for
mypy against redis-py's bundled types.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@chromy chromy changed the title test(client): Reproduce RunStorage.read_many crash on redis-py RedisCluster DNM: Reproduce RunStorage.read_many crash on redis-py RedisCluster Nov 4, 2026
@chromy chromy closed this Nov 4, 2026
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.

1 participant