Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Push replication from Android LiteServ and sync_gateway not pushing all deleted documents #1243

Closed
sethrosetter opened this issue May 26, 2016 · 3 comments

Comments

@sethrosetter
Copy link

commented May 26, 2016

Using Android 0.0.0-710 and sync_gateway 1.2.1-4

The LiteServ db has push / pull replication with a sync_gateway db setup.
I deleted documents on a a liteserv db and only a subset of the deleted document replicate.

Below is the failure step. It is interesting that the first time I delete docs on the LiteServ db, they all successfully replicate to sync_gateway. However after creating the docs with the same doc ids, and trying the same action, they do not all replicate.

Failure Step - https://github.com/couchbaselabs/mobile-testkit/blob/feature/implement-compact-revs-limit-test/testsuites/listener/shared/listener_sg_replication_compact_revs_limit.robot#L124

I have reviewed this with @hideki and will help him set up an reproduction next week.

@hideki hideki added the bug label Jun 2, 2016

@hideki hideki added the ready label Jun 23, 2016

@hideki hideki added this to the 1.3 milestone Jun 23, 2016

@hideki hideki added backlog and removed ready labels Jun 23, 2016

@sethrosetter

This comment has been minimized.

Copy link
Author

commented Jun 23, 2016

Need to verify but I think this was due to a misunderstanding - couchbase/couchbase-lite-ios#1277

@hideki hideki added review and removed backlog labels Jun 24, 2016

@hideki

This comment has been minimized.

Copy link

commented Jun 29, 2016

@sethrosetter Do you have any update for this ticket?

@sethrosetter

This comment has been minimized.

Copy link
Author

commented Jul 2, 2016

I am working on running these tests again with fixes to avoid the race condition pushing which I'm 99% sure was the reason for filing this. Once I get a chance to run this test again, I will update the results.

@sethrosetter sethrosetter removed the review label Jul 14, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.