From 0484dbcf68638149a8510537fd1a42c836f84020 Mon Sep 17 00:00:00 2001 From: "dejun.xdj" Date: Tue, 10 Jul 2018 16:46:48 +0800 Subject: [PATCH] Streams: using streamCompareID() instead of direct compare. --- src/t_stream.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/src/t_stream.c b/src/t_stream.c index 54d6b0d1..6a4439d0 100644 --- a/src/t_stream.c +++ b/src/t_stream.c @@ -1392,9 +1392,7 @@ void xreadCommand(client *c) { * synchronously in case the group top item delivered is smaller * than what the stream has inside. */ streamID *last = &groups[i]->last_id; - if (s->last_id.ms > last->ms || - (s->last_id.ms == last->ms && s->last_id.seq > last->seq)) - { + if (streamCompareID(&s->last_id, last) > 0) { serve_synchronously = 1; *gt = *last; } @@ -1402,9 +1400,7 @@ void xreadCommand(client *c) { } else { /* For consumers without a group, we serve synchronously if we can * actually provide at least one item from the stream. */ - if (s->last_id.ms > gt->ms || - (s->last_id.ms == gt->ms && s->last_id.seq > gt->seq)) - { + if (streamCompareID(&s->last_id, gt) > 0) { serve_synchronously = 1; } }