Skip to content

HIVE-25380: Remove the Hive Privilege Object for Database in the Read…#2525

Merged
yongzhi merged 2 commits intoapache:masterfrom
saihemanth-cloudera:hive-25380
Jul 27, 2021
Merged

HIVE-25380: Remove the Hive Privilege Object for Database in the Read…#2525
yongzhi merged 2 commits intoapache:masterfrom
saihemanth-cloudera:hive-25380

Conversation

@saihemanth-cloudera
Copy link
Contributor

…TableEvent and CreatTableEvent

What changes were proposed in this pull request?

Removing the Hive Privilege Object for Database in the ReadTableEvent and CreateTableEvent

Why are the changes needed?

Otherwise, we'll have to add an extra ranger policy for users to use any table within the database.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Local machine, Remote Cluster.

Copy link
Contributor

@yongzhi yongzhi left a comment

Choose a reason for hiding this comment

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

The change looks OK

@yongzhi yongzhi merged commit 773b929 into apache:master Jul 27, 2021
zchovan pushed a commit to zchovan/hive that referenced this pull request Jul 29, 2021
…TableEvent and CreateTableEvent (apache#2525) (Sai Hemanth Gantasala, reviewed by Yongzhi Chen)

* HIVE-25380: Remove the Hive Privilege Object for Database in the ReadTableEvent and CreatTableEvent

* Re-trigger ptests
dengzhhu653 pushed a commit to dengzhhu653/hive that referenced this pull request Dec 15, 2022
…TableEvent and CreateTableEvent (apache#2525) (Sai Hemanth Gantasala, reviewed by Yongzhi Chen)

* HIVE-25380: Remove the Hive Privilege Object for Database in the ReadTableEvent and CreatTableEvent

* Re-trigger ptests
PeterMoonAbsa pushed a commit to PeterMoonAbsa/hive that referenced this pull request Mar 12, 2025
…TableEvent and CreateTableEvent (apache#2525) (Sai Hemanth Gantasala, reviewed by Yongzhi Chen)

* HIVE-25380: Remove the Hive Privilege Object for Database in the ReadTableEvent and CreatTableEvent

* Re-trigger ptests

(cherry picked from commit 773b929)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants