Skip to content

Pass TranslogRecoveryRunner to engine from outside#33449

Merged
dnhatn merged 1 commit intoelastic:masterfrom
dnhatn:translog-handler
Sep 6, 2018
Merged

Pass TranslogRecoveryRunner to engine from outside#33449
dnhatn merged 1 commit intoelastic:masterfrom
dnhatn:translog-handler

Conversation

@dnhatn
Copy link
Copy Markdown
Member

@dnhatn dnhatn commented Sep 5, 2018

This commit allows us to use different TranslogRecoveryRunner when recovering an engine from its local translog. This change is a prerequisite for the commit-based rollback PR (#32867).

Relates: #32867 (comment)

This commit allows us to use different TranslogRecoveryRunner when recovering an engine from its local translog. This change is a prerequisite for the commit-based rollback PR (elastic#32867).

See elastic#32867 (comment)
@dnhatn dnhatn added >non-issue v7.0.0 :Distributed/Engine Anything around managing Lucene and the Translog in an open shard. v6.5.0 labels Sep 5, 2018
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed

@dnhatn dnhatn requested review from bleskes and s1monw September 5, 2018 22:59
@dnhatn dnhatn added the review label Sep 6, 2018
Copy link
Copy Markdown
Contributor

@s1monw s1monw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor

@bleskes bleskes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I'm curious to see what the other usage will be :)

@dnhatn
Copy link
Copy Markdown
Member Author

dnhatn commented Sep 6, 2018

Thanks @s1monw and @bleskes.

@dnhatn dnhatn merged commit 8afe09a into elastic:master Sep 6, 2018
@dnhatn dnhatn deleted the translog-handler branch September 6, 2018 15:59
dnhatn added a commit that referenced this pull request Sep 6, 2018
This commit allows us to use different TranslogRecoveryRunner when
recovering an engine from its local translog. This change is a
prerequisite for the commit-based rollback PR.

Relates #32867
@jimczi jimczi added v7.0.0-beta1 and removed v7.0.0 labels Feb 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Engine Anything around managing Lucene and the Translog in an open shard. >non-issue v6.5.0 v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants