hive icon indicating copy to clipboard operation
hive copied to clipboard

Published 20 hours ago • verified publisher icon apache

HIVE-25621: Added HivePrivilegeObjects for the commands Alter table m…

Open saihemanth-cloudera opened this issue 3 years ago • 6 comments

…erge/concatenate

What changes were proposed in this pull request?

Added HivePrivilegeObjects for alter table merge/concatenate commands.

Why are the changes needed?

To prevent malicious users from doing alter operations on table.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Local machine, remote cluster.

saihemanth-cloudera avatar Oct 18 '21 23:10 saihemanth-cloudera

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Feel free to reach out on the [email protected] list if the patch is in need of reviews.

github-actions[bot] avatar Jan 14 '22 00:01 github-actions[bot]

@dengzhhu653 - Can you please review my patch?

saihemanth-cloudera avatar Jul 20 '22 16:07 saihemanth-cloudera

@dengzhhu653 - Can you please review my patch again? Thanks

saihemanth-cloudera avatar Aug 22 '22 16:08 saihemanth-cloudera

@dengzhhu653 Can you review my patch again?

saihemanth-cloudera avatar Aug 30 '22 19:08 saihemanth-cloudera

@dengzhhu653 Can you review my patch again?

leave a minor comment, also the failed tests seem related, others look good to me.

dengzhhu653 avatar Aug 31 '22 07:08 dengzhhu653

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug C 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 9 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

sonarqubecloud[bot] avatar Sep 23 '22 16:09 sonarqubecloud[bot]

@dengzhhu653 - Tests are green now. Can you do another review here?

saihemanth-cloudera avatar Sep 26 '22 18:09 saihemanth-cloudera