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

tidb: update start ts for NewTxn. #4628

Merged
merged 3 commits into from
Sep 25, 2017
Merged

tidb: update start ts for NewTxn. #4628

merged 3 commits into from
Sep 25, 2017

Conversation

coocood
Copy link
Member

@coocood coocood commented Sep 25, 2017

The start_ts is only updated in ActivePendingTxn, it need to be updated in NexTxn as well.

The start_ts is only updated in `ActivePendingTxn`, it need to be updated in `NexTxn` as well.
@coocood coocood changed the title *: update start ts for NewTxn. tidb: update start ts for NewTxn. Sep 25, 2017
@shenli
Copy link
Member

shenli commented Sep 25, 2017

LGTM

@shenli
Copy link
Member

shenli commented Sep 25, 2017

/run-all-tests

@coocood coocood added the status/LGT1 Indicates that a PR has LGTM 1. label Sep 25, 2017
@coocood
Copy link
Member Author

coocood commented Sep 25, 2017

/run-all-test

@XuHuaiyu
Copy link
Contributor

LGTM

@XuHuaiyu XuHuaiyu added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Sep 25, 2017
@coocood coocood merged commit 0094512 into master Sep 25, 2017
@coocood coocood deleted the coocood/begin-start-ts branch September 25, 2017 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/LGT2 Indicates that a PR has LGTM 2.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants