Skip to content

executor: fix data race in GetDirtyTable() (#12767)#12900

Merged
sre-bot merged 4 commits intopingcap:release-2.1from
lzmhhh123:release-2.1-cherry-pick-12767
Nov 4, 2019
Merged

executor: fix data race in GetDirtyTable() (#12767)#12900
sre-bot merged 4 commits intopingcap:release-2.1from
lzmhhh123:release-2.1-cherry-pick-12767

Conversation

@lzmhhh123
Copy link
Contributor

cherry-pick #12767

@lzmhhh123 lzmhhh123 added type/bugfix This PR fixes a bug. sig/execution SIG execution type/2.1 cherry-pick labels Oct 23, 2019
Copy link
Contributor

@jackysp jackysp left a comment

Choose a reason for hiding this comment

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

LGTM

zz-jason
zz-jason previously approved these changes Oct 30, 2019
Copy link
Member

@zz-jason zz-jason left a comment

Choose a reason for hiding this comment

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

LGTM

@zz-jason zz-jason added status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2. labels Oct 30, 2019
@sre-bot
Copy link
Contributor

sre-bot commented Oct 30, 2019

Your auto merge job has been accepted, waiting for 13036, 13039, 13031, 12960, 12962

@sre-bot
Copy link
Contributor

sre-bot commented Oct 30, 2019

/run-all-tests

@sre-bot
Copy link
Contributor

sre-bot commented Oct 30, 2019

@lzmhhh123 merge failed.

@jackysp
Copy link
Contributor

jackysp commented Nov 4, 2019

/merge

@sre-bot
Copy link
Contributor

sre-bot commented Nov 4, 2019

/run-all-tests

@sre-bot sre-bot merged commit 418023f into pingcap:release-2.1 Nov 4, 2019
@lzmhhh123 lzmhhh123 deleted the release-2.1-cherry-pick-12767 branch December 24, 2019 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sig/execution SIG execution status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2. type/bugfix This PR fixes a bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants