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

CCR: Assert existing operation in following primary #34664

Closed
wants to merge 1 commit into from

Conversation

dnhatn
Copy link
Member

@dnhatn dnhatn commented Oct 20, 2018

Follow-up: #34408 (comment)

This PR adds an assertion which asserts that the existing operation is
equal to the processing operation except for the primary term.

Relates #34412

This PR adds an assertion which asserts that the existing operation is
equal to the processing operation except for the primary term.

Relates elastic#34412
@dnhatn dnhatn added >non-issue :Distributed Indexing/CCR Issues around the Cross Cluster State Replication features labels Oct 20, 2018
@dnhatn dnhatn requested a review from bleskes October 20, 2018 01:50
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

@dnhatn dnhatn changed the title CC: Assert existing operation in following primary CCR: Assert existing operation in following primary Oct 20, 2018
@dnhatn
Copy link
Member Author

dnhatn commented Dec 27, 2018

@bleskes I am closing this PR for I think it is no longer required.

@dnhatn dnhatn closed this Dec 27, 2018
@dnhatn dnhatn deleted the following-assert-ops branch December 27, 2018 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed Indexing/CCR Issues around the Cross Cluster State Replication features >non-issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants