[feat](authorization)Centralizing Common Authorization Operations in Common Interface#43595
Closed
CalvinKirs wants to merge 1 commit intoapache:masterfrom
Closed
[feat](authorization)Centralizing Common Authorization Operations in Common Interface#43595CalvinKirs wants to merge 1 commit intoapache:masterfrom
CalvinKirs wants to merge 1 commit intoapache:masterfrom
Conversation
…a Common Interface ### Optimize Column-Level Permission Checks Using Table-Level Permissions: Since having column-level permissions does not imply table-level permissions, but having table-level permissions does imply permissions on all columns within the table, we can streamline column permission checks. When checking column-level permissions, we can first check if the user has table-level permissions. If table-level permissions are granted, column-level checks become unnecessary. Only if table-level permissions are absent do we proceed with specific column-level permission checks. ### Global Permissions Shortcut: Global-level permissions typically grant full access across all operations. Therefore, to optimize permission checks, we can add an early check for global permissions. If the user has global permissions, they are authorized, and further permission checks at the database, table, or column levels are unnecessary, allowing us to return immediately.
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
bobhan1
pushed a commit
to bobhan1/doris
that referenced
this pull request
Nov 14, 2024
…Common Interface apache#43595 (apache#208) * [feat](authorization)Centralizing Common Authorization Operations in Common Interface apache#43595 * [feat](authorization)Centralizing Common Authorization Operations in Common Interface apache#43595
Contributor
|
We're closing this PR because it hasn't been updated in a while. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What problem does this PR solve?
Optimize Column-Level Permission Checks Using Table-Level Permissions:
Since having column-level permissions does not imply table-level permissions, but having table-level permissions does imply permissions on all columns within the table, we can streamline column permission checks. When checking column-level permissions, we can first check if the user has table-level permissions. If table-level permissions are granted, column-level checks become unnecessary. Only if table-level permissions are absent do we proceed with specific column-level permission checks.
###$ Global Permissions Shortcut: Global-level permissions typically grant full access across all operations.
Therefore, to optimize permission checks, we can add an early check for global permissions. If the user has global permissions, they are authorized, and further permission checks at the database, table, or column levels are unnecessary, allowing us to return immediately.
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)