On Jul 11, 2014, at 9:12 AM, Traun Leyden <[email protected]> wrote:

> it seems like the if/then check is moot, due to 
>     _caughtUp = NO;
> always being called before this line.

You're right. That 'if' test is a remnant of some logic that moved into 
CBLSocketChangeTracker (lines 212-213). In a continuous pull replication, when 
WebSockets aren't available, we first get the changes feed using feed=normal, 
then after that switch to feed=longpoll. (The reason is so the puller can 
quickly tell that it's already up to date, instead of waiting indefinitely for 
the longpoll feed to return something.)

I'll make the change in the Obj-C repo.

--Jens

-- 
You received this message because you are subscribed to the Google Groups 
"Couchbase Mobile" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/mobile-couchbase/9ECCEE60-BCBF-477D-96BB-F5CF4C00E265%40couchbase.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to