Skip to content

Commit ea3b4ab

Browse files
committed
Add comment about temporary fakeredis fork dependency
Created upstream PR cunla/fakeredis-py#427 to fix xpending_range. Using fork temporarily until PR is merged and released. The fix is required because fakeredis was only returning 2 fields per pending message (message_id, consumer) when Redis spec requires 4 fields (message_id, consumer, time_since_delivered, times_delivered). This was causing KeyError in docket's snapshot() method which accesses these fields.
1 parent a0b98ac commit ea3b4ab

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

pyproject.toml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,9 @@ classifiers = [
2121
]
2222
dependencies = [
2323
"cloudpickle>=3.1.1",
24+
# TEMPORARY: Using fork until https://github.com/cunla/fakeredis-py/pull/427 is merged
25+
# This fixes xpending_range to return all 4 required fields (message_id, consumer,
26+
# time_since_delivered, times_delivered) instead of just 2, matching Redis behavior
2427
"fakeredis[lua] @ git+https://github.com/zzstoatzz/fakeredis-py.git@fix-xpending-range-fields",
2528
"opentelemetry-api>=1.30.0",
2629
"opentelemetry-exporter-prometheus>=0.51b0",

0 commit comments

Comments
 (0)