diff options
| author | antirez <antirez@gmail.com> | 2018-11-19 17:00:30 +0100 |
|---|---|---|
| committer | antirez <antirez@gmail.com> | 2018-11-19 17:00:34 +0100 |
| commit | 2bd6802fa1bd48260022435e9a040b8233d6cdfc (patch) | |
| tree | 5913a3bddcf660b8435231294184dcd16309a350 /src/t_stream.c | |
| parent | 18d0759b14cee97430c08af3d6674929b62390a0 (diff) | |
| download | redis-2bd6802fa1bd48260022435e9a040b8233d6cdfc.tar.gz | |
Stream: fix XREADGROUP history reading of deleted messages.
This commit fixes #5570. It is a similar bug to one fixed a few weeks
ago and is due to the range API to be called with NULL as "end ID"
parameter instead of repeating again the start ID, to be sure that we
selectively issue the entry with a given ID, or we get zero returned
(and we know we should emit a NULL reply).
Diffstat (limited to 'src/t_stream.c')
| -rw-r--r-- | src/t_stream.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/t_stream.c b/src/t_stream.c index dd83f2b1d..f51f6c46b 100644 --- a/src/t_stream.c +++ b/src/t_stream.c @@ -1025,7 +1025,7 @@ size_t streamReplyWithRangeFromConsumerPEL(client *c, stream *s, streamID *start if (end && memcmp(ri.key,end,ri.key_len) > 0) break; streamID thisid; streamDecodeID(ri.key,&thisid); - if (streamReplyWithRange(c,s,&thisid,NULL,1,0,NULL,NULL, + if (streamReplyWithRange(c,s,&thisid,&thisid,1,0,NULL,NULL, STREAM_RWR_RAWENTRIES,NULL) == 0) { /* Note that we may have a not acknowledged entry in the PEL |