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

INTEGRATION [PR#594 > development/8.1] ft: ZENKO-1175 tailable cursor to consume mongo oplog #600

Merged
merged 3 commits into from
Nov 8, 2018

Conversation

bert-e
Copy link
Contributor

@bert-e bert-e commented Nov 8, 2018

This pull request has been created automatically.
It is linked to its parent pull request #594.

Do not edit this pull request directly.
If you need to amend/cancel the changeset on branch
w/8.1/bugfix/ZENKO-1175-oplogSkipFix, please follow this
procedure:

 $ git fetch
 $ git checkout w/8.1/bugfix/ZENKO-1175-oplogSkipFix
 $ # <amend or cancel the changeset by _adding_ new commits>
 $ git push origin w/8.1/bugfix/ZENKO-1175-oplogSkipFix

Please always comment pull request #594 instead of this one.

jonathan-gramain and others added 3 commits November 8, 2018 14:45
Use a tailable custor to keep ordering guarantees for the records we
read. This also means we have to read from the beginning when we
reconnect (at startup), and start processing when we encountered the
unique ID previously stored in zookeeper.

Also removed dispatcher mode with MongoLogReader (was only used for
the short-lived Federation deployment of Zenko).
In case where there is no saved ID yet (initial deployment), do
process the very first entry in the log instead of skipping it. In
practice it should not have an impact because the very first entry in
the log is normally not due to be processed for CRR, but it ensures
correctness.
@bert-e bert-e merged commit 7f54136 into development/8.1 Nov 8, 2018
@bert-e bert-e deleted the w/8.1/bugfix/ZENKO-1175-oplogSkipFix branch November 8, 2018 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants