Skip to content

executor, privilege: introduce a new privilege for the set config statement#16847

Merged
sre-bot merged 7 commits intopingcap:masterfrom
qw4990:conf-priv
Apr 27, 2020
Merged

executor, privilege: introduce a new privilege for the set config statement#16847
sre-bot merged 7 commits intopingcap:masterfrom
qw4990:conf-priv

Conversation

@qw4990
Copy link
Contributor

@qw4990 qw4990 commented Apr 26, 2020

What problem does this PR solve?

Issue Number: close #16846

Problem Summary: introduce a new privilege for the set config statement instead of using the SuperPriv.

What is changed and how it works?

Wait for pingcap/parser#835.

  1. Using the new privilege ConfigPriv.
  2. Add and change some tests.

Check List

Tests

  • Unit test

fixup

fixup

add tests

fixup

fixup

fixup
@codecov
Copy link

codecov bot commented Apr 27, 2020

Codecov Report

Merging #16847 into master will decrease coverage by 0.0556%.
The diff coverage is 75.0000%.

@@               Coverage Diff                @@
##             master     #16847        +/-   ##
================================================
- Coverage   80.3938%   80.3382%   -0.0557%     
================================================
  Files           507        507                
  Lines        137656     137363       -293     
================================================
- Hits         110667     110355       -312     
- Misses        18327      18347        +20     
+ Partials       8662       8661         -1     

Copy link
Member

@zz-jason zz-jason left a comment

Choose a reason for hiding this comment

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

LGTM

@tiancaiamao
Copy link
Contributor

LGTM

Do not forget the special handle of skip-grant-table config.

@tiancaiamao tiancaiamao added the status/LGT2 Indicates that a PR has LGTM 2. label Apr 27, 2020
@tiancaiamao tiancaiamao added the status/can-merge Indicates a PR has been approved by a committer. label Apr 27, 2020
@sre-bot
Copy link
Contributor

sre-bot commented Apr 27, 2020

/run-all-tests

@sre-bot sre-bot merged commit 468b8c6 into pingcap:master Apr 27, 2020
sre-bot pushed a commit to sre-bot/tidb that referenced this pull request Apr 27, 2020
Signed-off-by: sre-bot <sre-bot@pingcap.com>
@sre-bot
Copy link
Contributor

sre-bot commented Apr 27, 2020

cherry pick to release-4.0 in PR #16856

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/privilege sig/execution SIG execution status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

executor: Introduce a new privilege for the set config statement

4 participants