executor: adapt the GRANT and REVOKE for column privilege
What problem does this PR solve?
Issue Number: ref #61706
Problem Summary:
What changed and how does it work?
Adapt the GRANT and REVOKE for column privilege
This PR pick pkg/executor/grant.go, pkg/executor/revoke.go, pkg/executor/revoke_test.go and pkg/executor/utils.go in https://github.com/pingcap/tidb/pull/61638
Check List
Tests
- [x] Unit test
- [ ] Integration test
- [ ] Manual test (add detailed scripts or steps below)
- [ ] No need to test
- [ ] I checked and no code files have been changed.
Side effects
- [ ] Performance regression: Consumes more CPU
- [ ] Performance regression: Consumes more Memory
- [ ] Breaking backward compatibility
Documentation
- [ ] Affects user behaviors
- [ ] Contains syntax changes
- [ ] Contains variable changes
- [ ] Contains experimental features
- [ ] Changes MySQL compatibility
Release note
Please refer to Release Notes Language Style Guide to write a quality release note.
None
Codecov Report
Attention: Patch coverage is 74.68354% with 60 lines in your changes missing coverage. Please review.
Please upload report for BASE (
release-8.5-20250606-v8.5.2@4f65dff). Learn more about missing BASE report.
Additional details and impacted files
@@ Coverage Diff @@
## release-8.5-20250606-v8.5.2 #61678 +/- ##
================================================================
Coverage ? 57.5797%
================================================================
Files ? 1770
Lines ? 629154
Branches ? 0
================================================================
Hits ? 362265
Misses ? 242363
Partials ? 24526
| Flag | Coverage Δ | |
|---|---|---|
| integration | 38.0301% <62.4472%> (?) |
|
| unit | 72.7110% <73.8396%> (?) |
Flags with carried forward coverage won't be shown. Click here to find out more.
| Components | Coverage Δ | |
|---|---|---|
| dumpling | 52.9278% <0.0000%> (?) |
|
| parser | ∅ <0.0000%> (?) |
|
| br | 63.1635% <0.0000%> (?) |
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
/retest
/retest
@ystaticy: adding LGTM is restricted to approvers and reviewers in OWNERS files.
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.
/hold
/unhold
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: tiancaiamao, wjhuang2016, ystaticy
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~OWNERS~~ [tiancaiamao,wjhuang2016]
- ~~pkg/domain/OWNERS~~ [wjhuang2016]
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
[LGTM Timeline notifier]
Timeline:
2025-06-19 07:03:55.661697924 +0000 UTC m=+341688.384876906: :ballot_box_with_check: agreed by tiancaiamao.2025-06-19 12:06:41.299637224 +0000 UTC m=+359854.022816208: :ballot_box_with_check: agreed by wjhuang2016.