-
Notifications
You must be signed in to change notification settings - Fork 679
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
optimistic-transaction: remove reference to being "default" #3146
Conversation
No longer accurate in 3.0+
Co-authored-by: Jack Yu <yusp@pingcap.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@jackysp,Thanks for your review. |
Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com>
Sorry @nullnotnil, you don't have permission to trigger auto merge event on this branch. You are not a committer or co-leader or leader for the related sigs:docs(slack). |
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
cherry pick to release-3.0 in PR #3202 |
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
cherry pick to release-3.1 in PR #3203 |
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
cherry pick to release-4.0 in PR #3204 |
Signed-off-by: ti-srebot <ti-srebot@pingcap.com> Co-authored-by: Null not nil <67764674+nullnotnil@users.noreply.github.com>
/unlabel translation/doing |
/label translation/done |
What is changed, added or deleted? (Required)
The documentation claimed optimistic transactions are the default. This is only true for v2.1.
Which TiDB version(s) do your changes apply to? (Required)
What is the related PR or file link(s)?