Skip to content
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

Authz plugin support path-based table #5368

Closed
2 of 3 tasks
pan3793 opened this issue Oct 8, 2023 · 11 comments
Closed
2 of 3 tasks

Authz plugin support path-based table #5368

pan3793 opened this issue Oct 8, 2023 · 11 comments

Comments

@pan3793
Copy link
Member

pan3793 commented Oct 8, 2023

Code of Conduct

Search before creating

  • I have searched in the task list and found no similar tasks.

Mentor

  • I have sufficient knowledge and experience of this task, and I volunteer to be the mentor of this task to guide contributors to complete the task.

Skill requirements

  • Knowledge about Spark Catalyst
  • Basic Knowledge of SQL authorization model, e.g. Apache Ranger

Background and Goals

Allow the authz plugin to manage permissions for path-based tables, as the catalog table does.
#5166

Implementation steps

Refer existing code of authz module, which is basically a Spark SQL extension, powers the Spark SQL with authorization capabilities by injecting some catalyst rules.

Additional context

Original report is @yaooqinn

@Emor-nj
Copy link
Contributor

Emor-nj commented Oct 10, 2023

Hi @pan3793, can you assign this task to me? I will try to finish it before November 11th. Thanks.

@pan3793
Copy link
Member Author

pan3793 commented Oct 10, 2023

assigned, thanks

@pan3793
Copy link
Member Author

pan3793 commented Oct 24, 2023

@Emor-nj do you have any progress on this task?

@pan3793
Copy link
Member Author

pan3793 commented Oct 25, 2023

revoked the assignment since there has been no response or update for a long time.

@pan3793
Copy link
Member Author

pan3793 commented Oct 25, 2023

@AngersZhuuuu I see this task blocks your works, would you like to take up this one too?

@AngersZhuuuu
Copy link
Contributor

Yea

@AngersZhuuuu
Copy link
Contributor

InsertIntoDataSourceCommand
InsertIntoHiveDirCommand
SaveIntoDataSourceCommand

@AngersZhuuuu
Copy link
Contributor

AngersZhuuuu commented Nov 2, 2023

CreateDatabaseCommand
AlterDatabaseSetLocationCommand

CreateTable
AlterTableAddPartitionCommand
AlterTableSetLocationCommand

AddJarsCommand
AddFilesCommand
AddArchivesCommand
ListFilesCommand
ListJarsCommand
ListArchivesCommand

LoadDataCommand

@yaooqinn
Copy link
Member

yaooqinn commented Nov 3, 2023

Spark has forbidden alter locs for AlterDatabasePropertiesCommand / AlterTableSetPropertiesCommand, right?

@AngersZhuuuu
Copy link
Contributor

Spark has forbidden alter locs for AlterDatabasePropertiesCommand / AlterTableSetPropertiesCommand, right?

Yea

@pan3793
Copy link
Member Author

pan3793 commented Dec 11, 2023

This task was completed by splitting into several small tasks.

@pan3793 pan3793 changed the title 、Authz plugin support path-based table Authz plugin support path-based table Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants