-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[Feature](privilege) support data mask in doris internal auth #38301
base: master
Are you sure you want to change the base?
Conversation
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
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.
Can you please add some PR descriptions
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/logical/LogicalCheckPolicy.java
Outdated
Show resolved
Hide resolved
7a6fd8b
to
9ae811e
Compare
run buildall |
TPC-H: Total hot run time: 38388 ms
|
run buildall |
TPC-H: Total hot run time: 37996 ms
|
TPC-DS: Total hot run time: 197140 ms
|
ClickBench: Total hot run time: 31.78 s
|
run buildall |
TPC-H: Total hot run time: 38002 ms
|
TPC-DS: Total hot run time: 196600 ms
|
ClickBench: Total hot run time: 31.64 s
|
4ce03c7
to
397f3a9
Compare
run buildall |
fe/fe-core/src/main/antlr4/org/apache/doris/nereids/DorisParser.g4
Outdated
Show resolved
Hide resolved
fe/fe-core/src/main/antlr4/org/apache/doris/nereids/DorisParser.g4
Outdated
Show resolved
Hide resolved
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/CreatePolicyCommand.java
Outdated
Show resolved
Hide resolved
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/DropPolicyCommand.java
Outdated
Show resolved
Hide resolved
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/ShowPolicyCommand.java
Outdated
Show resolved
Hide resolved
fe/fe-core/src/main/java/org/apache/doris/nereids/util/RelationUtil.java
Outdated
Show resolved
Hide resolved
fe/fe-core/src/main/java/org/apache/doris/mysql/privilege/InternalAccessController.java
Outdated
Show resolved
Hide resolved
fe/fe-core/src/main/java/org/apache/doris/analysis/ColumnName.java
Outdated
Show resolved
Hide resolved
fe/fe-core/src/main/java/org/apache/doris/policy/DorisDataMaskPolicy.java
Outdated
Show resolved
Hide resolved
3df41ac
to
ab922a9
Compare
run buildall |
TPC-H: Total hot run time: 38256 ms
|
run buildall |
TPC-H: Total hot run time: 37850 ms
|
TPC-DS: Total hot run time: 196962 ms
|
ClickBench: Total hot run time: 31.57 s
|
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
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
run P0 |
run buildall |
TPC-H: Total hot run time: 38045 ms
|
TPC-DS: Total hot run time: 193079 ms
|
ClickBench: Total hot run time: 31.8 s
|
run cloud_p0 |
PR approved by at least one committer and no changes requested. |
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 buildall |
run buildall |
1 similar comment
run buildall |
TPC-H: Total hot run time: 40901 ms
|
TPC-DS: Total hot run time: 192868 ms
|
ClickBench: Total hot run time: 33.06 s
|
run cloud_p0 |
c38ecbf
to
df30025
Compare
df30025
to
c7276bc
Compare
24b7518
to
8f011cd
Compare
run buildall |
TPC-H: Total hot run time: 31662 ms
|
TPC-DS: Total hot run time: 183244 ms
|
ClickBench: Total hot run time: 30.4 s
|
run p0 |
Proposed changes
Before, data masking could only be managed by ranger, and ranger was not applicable to users in most cases. doris built-in permission management did not have the management function of data masking, and now it is added.
supported data mask policy
Issue Number: close #xxx