[ https://issues.apache.org/jira/browse/HIVE-25621?focusedWorklogId=803007&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-803007 ]
ASF GitHub Bot logged work on HIVE-25621: ----------------------------------------- Author: ASF GitHub Bot Created on: 23/Aug/22 19:20 Start Date: 23/Aug/22 19:20 Worklog Time Spent: 10m Work Description: saihemanth-cloudera commented on code in PR #2731: URL: https://github.com/apache/hive/pull/2731#discussion_r953021259 ########## ql/src/java/org/apache/hadoop/hive/ql/ddl/table/storage/compact/AlterTableCompactAnalyzer.java: ########## @@ -67,6 +73,18 @@ protected void analyzeCommand(TableName tableName, Map<String, String> partition } AlterTableCompactDesc desc = new AlterTableCompactDesc(tableName, partitionSpec, type, isBlocking, mapProp); +// Table table = getTable(tableName); Review Comment: Sure ########## ql/src/java/org/apache/hadoop/hive/ql/ddl/table/storage/compact/AlterTableCompactAnalyzer.java: ########## @@ -27,7 +27,13 @@ import org.apache.hadoop.hive.ql.ddl.DDLWork; import org.apache.hadoop.hive.ql.ddl.DDLSemanticAnalyzerFactory.DDLType; import org.apache.hadoop.hive.ql.ddl.table.AbstractAlterTableAnalyzer; +import org.apache.hadoop.hive.ql.ddl.table.AlterTableType; import org.apache.hadoop.hive.ql.exec.TaskFactory; +import org.apache.hadoop.hive.ql.hooks.ReadEntity; Review Comment: Ack Issue Time Tracking ------------------- Worklog Id: (was: 803007) Time Spent: 2h 20m (was: 2h 10m) > Alter table partition compact/concatenate commands should send > HivePrivilegeObjects for Authz > --------------------------------------------------------------------------------------------- > > Key: HIVE-25621 > URL: https://issues.apache.org/jira/browse/HIVE-25621 > Project: Hive > Issue Type: Bug > Affects Versions: 4.0.0 > Reporter: Sai Hemanth Gantasala > Assignee: Sai Hemanth Gantasala > Priority: Major > Labels: pull-request-available > Time Spent: 2h 20m > Remaining Estimate: 0h > > # Run the following queries > Create table temp(c0 int) partitioned by (c1 int); > Insert into temp values(1,1); > ALTER TABLE temp PARTITION (c1=1) COMPACT 'minor'; > ALTER TABLE temp PARTITION (c1=1) CONCATENATE; > Insert into temp values(1,1); > # The above compact/concatenate commands are currently not sending any hive > privilege objects for authorization. Hive needs to send these objects to > avoid malicious users doing any operation. -- This message was sent by Atlassian Jira (v8.20.10#820010)