-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
executor: stop using the deprecated Filter field in BRIE statements #18172
Conversation
PTAL @wjhuang2016 |
Data race
|
/run-unit-test |
Codecov Report
@@ Coverage Diff @@
## master #18172 +/- ##
===========================================
Coverage 79.5462% 79.5462%
===========================================
Files 535 535
Lines 144712 144712
===========================================
Hits 115113 115113
Misses 20324 20324
Partials 9275 9275 |
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
PTAL @bb7133 |
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
/merge |
/merge |
/run-all-tests |
@kennytm merge failed. |
/run-all-tests |
/run-integration-copr-test |
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
cherry pick to release-4.0 in PR #18764 |
…ingcap#18172) Co-authored-by: ti-srebot <66930949+ti-srebot@users.noreply.github.com>
What problem does this PR solve?
Issue Number: close #18151
Problem Summary:
In pingcap/br#313, we have deprecated the fields
Filter
andCaseSensitive
in theConfig
structure (they are not immediately removed due to cyclic dependency). Their usages are supposed to be removed in the when TiDB upgrades the BR dependency.The Go language founders hate warnings, and thus the Go compiler will not warn about use of deprecated items. This has to be checked by third-party linters such as staticcheck. Unforuntately SA1019 is not enabled by default. Therefore, the lint is completely ignored in #17839. Because BRIE statement does not support mocktikv, we can't rely on unit tests either. This leads to #18151 slipping into the master branch.
What is changed and how it works?
What's Changed:
As a hot-fix, we removed all use of Filter and CaseSensitive fields by the TableFilter field.
How it Works:
Related changes
Check List
Tests
BACKUP DATABASE test TO 'noop://'
in both master and this PR.Side effects
Release note