Skip to content

store/tikv: tiny refactoring, change twoPhaseCommitAction to interface (#12845)#12988

Merged
sre-bot merged 2 commits intopingcap:release-3.0from
tiancaiamao:refactor-cherry-3.0
Oct 30, 2019
Merged

store/tikv: tiny refactoring, change twoPhaseCommitAction to interface (#12845)#12988
sre-bot merged 2 commits intopingcap:release-3.0from
tiancaiamao:refactor-cherry-3.0

Conversation

@tiancaiamao
Copy link
Contributor

Cherry-pick #12845

@tiancaiamao tiancaiamao added type/3.0 cherry-pick type/enhancement The issue or PR belongs to an enhancement. labels Oct 29, 2019
@tiancaiamao
Copy link
Contributor Author

PTAL @lysu @jackysp @cfzjywxk

@coocood
Copy link
Member

coocood commented Oct 29, 2019

LGTM

1 similar comment
@cfzjywxk
Copy link
Contributor

LGTM

@tiancaiamao tiancaiamao added the status/LGT2 Indicates that a PR has LGTM 2. label Oct 29, 2019
@jackysp jackysp added the status/can-merge Indicates a PR has been approved by a committer. label Oct 29, 2019
@sre-bot
Copy link
Contributor

sre-bot commented Oct 29, 2019

/run-all-tests

@sre-bot
Copy link
Contributor

sre-bot commented Oct 29, 2019

@tiancaiamao merge failed.

@lysu
Copy link
Contributor

lysu commented Oct 29, 2019

/merge

@sre-bot
Copy link
Contributor

sre-bot commented Oct 29, 2019

Sorry @lysu, you don't have permission to trigger auto merge event on this branch.

@lysu
Copy link
Contributor

lysu commented Oct 29, 2019

/rebuild

@jackysp
Copy link
Contributor

jackysp commented Oct 30, 2019

/merge

@sre-bot
Copy link
Contributor

sre-bot commented Oct 30, 2019

/run-all-tests

@sre-bot sre-bot merged commit bdc59e6 into pingcap:release-3.0 Oct 30, 2019
@tiancaiamao tiancaiamao deleted the refactor-cherry-3.0 branch May 26, 2020 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2. type/enhancement The issue or PR belongs to an enhancement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants