-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
*: fix rolling_update_fail when using Arrow format. #12899
Conversation
2fa9e14
to
2185ed1
Compare
Codecov Report
@@ Coverage Diff @@
## master #12899 +/- ##
===========================================
Coverage 80.0299% 80.0299%
===========================================
Files 468 468
Lines 109424 109424
===========================================
Hits 87572 87572
Misses 15138 15138
Partials 6714 6714 |
/run-all-tests |
/run-all-tests tikv=release-3.0 |
2185ed1
to
22fed4b
Compare
/run-all-tests tikv=release-3.0 |
/run-all-tests tikv=release-3.0 tidb-test=release-3.0 pd=release-3.0 |
22fed4b
to
2ff5fc3
Compare
/run-all-tests tikv=release-3.0 tidb-test=release-3.0 pd=release-3.0 |
/run-all-tests tikv=release-3.0 |
2ff5fc3
to
3174116
Compare
/run-all-tests tikv=release-3.0 pd=release-3.0 |
/run-integration-common-test tikv=release-3.0 pd=release-3.0 |
3174116
to
fb6b58d
Compare
/run-integration-common-test tikv=release-3.0 pd=release-3.0 |
fb6b58d
to
c5e1cb7
Compare
/run-all-tests tikv=pr/5754 |
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
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
/run-all-tests tikv=pr/5754 |
CI failed? |
/run-all-tests tikv=pr/5754 |
@breeswish The fail case in CI has nothing to do with this commit. |
What problem does this PR solve?
In pingcap/tipb#135 , I change the photo and use the same number to indicate it. It will cause a compatibility problem, and rolling_update will fail.
What is changed and how it works?
Use a new number to indicate the new field.
Check List
Tests
Code changes
Side effects
Related changes
Release note