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

Update client-c to fix successive StoreNotMatch error (#1713) #1714

Merged
merged 4 commits into from
Apr 9, 2021

Conversation

ti-srebot
Copy link
Collaborator

@ti-srebot ti-srebot commented Mar 31, 2021

cherry-pick #1713 to release-5.0
You can switch your code base to this Pull Request by using git-extras:

# In tics repo:
git pr https://github.com/pingcap/tics/pull/1714

After apply modifications, you can push your change to this PR via:

git push git@github.com:ti-srebot/tics.git pr/1714:release-5.0-6bcd6e1a9df3

What problem does this PR solve?

Problem Summary:
See pingcap/tidb#22909

What is changed and how it works?

Proposal: xxx

What's Changed:

Related changes

  • Need to cherry-pick to the release branch:

Check List

Tests

  • No code

Release note

  • No release note

Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot ti-srebot added CHERRY-PICK cherry pick status/LGT1 Indicates that a PR has LGTM 1. type/bugfix This PR fixes a bug. labels Mar 31, 2021
@ti-srebot ti-srebot requested a review from solotzg March 31, 2021 11:26
@ti-srebot ti-srebot added this to the v5.0.0 ga milestone Mar 31, 2021
@JaySon-Huang JaySon-Huang modified the milestones: v5.0.0 ga, v5.0.1 Apr 7, 2021
Copy link
Contributor

@solotzg solotzg left a comment

Choose a reason for hiding this comment

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

lgtm

@solotzg solotzg added the status/can-merge Indicates a PR has been approved by a committer. label Apr 9, 2021
@ti-srebot
Copy link
Collaborator Author

/run-all-tests

@ti-srebot
Copy link
Collaborator Author

@ti-srebot merge failed.

@solotzg
Copy link
Contributor

solotzg commented Apr 9, 2021

/merge

@ti-srebot
Copy link
Collaborator Author

/run-all-tests

@solotzg
Copy link
Contributor

solotzg commented Apr 9, 2021

/rebuild

@ti-srebot
Copy link
Collaborator Author

/run-all-tests

@ti-srebot
Copy link
Collaborator Author

@ti-srebot merge failed.

@solotzg
Copy link
Contributor

solotzg commented Apr 9, 2021

/merge

@ti-srebot
Copy link
Collaborator Author

/run-all-tests

@ti-srebot ti-srebot merged commit f626917 into pingcap:release-5.0 Apr 9, 2021
@solotzg solotzg deleted the release-5.0-6bcd6e1a9df3 branch April 9, 2021 15:01
@solotzg solotzg linked an issue Apr 12, 2021 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CHERRY-PICK cherry pick status/can-merge Indicates a PR has been approved by a committer. status/LGT1 Indicates that a PR has LGTM 1. type/bugfix This PR fixes a bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

client-c can not deal with StoreNotMatch correctly
4 participants