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

Preserve is_retry_request flag for prewrite when retrying with region errors #513

Merged
merged 4 commits into from
May 31, 2022

Conversation

ekexium
Copy link
Contributor

@ekexium ekexium commented May 30, 2022

No description provided.

Signed-off-by: ekexium <ekexium@gmail.com>
@ekexium ekexium force-pushed the verify-is-retry branch from 49030e4 to 95bc01b Compare May 30, 2022 10:18
@ekexium ekexium marked this pull request as ready for review May 31, 2022 06:01
@ekexium ekexium changed the title [DNM] add a test to verify the incorrect setting of is_retry_request Preserve is_retry_request flag for prewrite May 31, 2022
@ekexium ekexium changed the title Preserve is_retry_request flag for prewrite Preserve is_retry_request flag for prewrite when retrying with region errors May 31, 2022
Signed-off-by: ekexium <ekexium@gmail.com>
@ekexium ekexium force-pushed the verify-is-retry branch from 2a8a8f6 to 5635dbc Compare May 31, 2022 06:03
@ekexium
Copy link
Contributor Author

ekexium commented May 31, 2022

PTAL @sticnarf @cfzjywxk

Signed-off-by: ekexium <ekexium@gmail.com>
Signed-off-by: ekexium <ekexium@gmail.com>
@sticnarf sticnarf merged commit e69cd80 into tikv:master May 31, 2022
sticnarf pushed a commit that referenced this pull request May 31, 2022
… errors (#513)

* add a test to verify the incorrect setting of is_retry_request

Signed-off-by: ekexium <ekexium@gmail.com>

* preserve the retry flag for prewrite

Signed-off-by: ekexium <ekexium@gmail.com>

* fix test

Signed-off-by: ekexium <ekexium@gmail.com>

* add more explanations

Signed-off-by: ekexium <ekexium@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants