-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23674/
-----------------------------------------------------------
(Updated Aug. 11, 2014, 12:53 a.m.)
Review request for hive and Thejas Nair.
Changes
-------
Supports renaming table cross database.
Bugs: HIVE-4064
https://issues.apache.org/jira/browse/HIVE-4064
Repository: hive-git
Description
-------
Hive doesn't consistently handle db qualified names across all HiveQL
statements. While some HiveQL statements such as SELECT support DB qualified
names, other such as CREATE INDEX doesn't.
Diffs (updated)
-----
itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/security/authorization/plugin/TestHiveAuthorizerCheckInvocation.java
c91b15c
itests/util/src/main/java/org/apache/hadoop/hive/ql/hooks/CheckColumnAccessHook.java
14fc430
metastore/src/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java
ea866c5
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
6e689d0
metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java
5a56ced
metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java 760777a
metastore/src/java/org/apache/hadoop/hive/metastore/Warehouse.java 74b1432
ql/src/java/org/apache/hadoop/hive/ql/Driver.java ea6ddbf
ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 376e040
ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java d22b1f6
ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 39b032e
ql/src/java/org/apache/hadoop/hive/ql/optimizer/IndexUtils.java 2e32fee
ql/src/java/org/apache/hadoop/hive/ql/optimizer/index/RewriteGBUsingIndex.java
989d0b5
ql/src/java/org/apache/hadoop/hive/ql/parse/BaseSemanticAnalyzer.java 22945e3
ql/src/java/org/apache/hadoop/hive/ql/parse/ColumnAccessInfo.java 939dc65
ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 67a3aa7
ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g ab1188a
ql/src/java/org/apache/hadoop/hive/ql/parse/IndexUpdater.java 856ec2f
ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 7b86414
ql/src/java/org/apache/hadoop/hive/ql/parse/authorization/HiveAuthorizationTaskFactoryImpl.java
826bdf3
ql/src/java/org/apache/hadoop/hive/ql/plan/AlterIndexDesc.java 0318e4b
ql/src/java/org/apache/hadoop/hive/ql/plan/AlterTableAlterPartDesc.java
cf67e16
ql/src/java/org/apache/hadoop/hive/ql/plan/AlterTableSimpleDesc.java 541675c
ql/src/java/org/apache/hadoop/hive/ql/plan/PrivilegeObjectDesc.java 9417220
ql/src/java/org/apache/hadoop/hive/ql/plan/RenamePartitionDesc.java 1b5fb9e
ql/src/java/org/apache/hadoop/hive/ql/plan/ShowColumnsDesc.java fe6a91e
ql/src/java/org/apache/hadoop/hive/ql/plan/ShowGrantDesc.java aa88153
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/AuthorizationUtils.java
5c94217
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HivePrivilegeObject.java
9e9ef71
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveV1Authorizer.java
fbc0090
ql/src/test/org/apache/hadoop/hive/ql/metadata/TestHive.java 98c2924
ql/src/test/org/apache/hadoop/hive/ql/parse/TestQBCompact.java 5f32d5f
ql/src/test/org/apache/hadoop/hive/ql/parse/authorization/PrivilegesTestBase.java
93901ec
ql/src/test/org/apache/hadoop/hive/ql/parse/authorization/TestHiveAuthorizationTaskFactory.java
ab0d80e
ql/src/test/org/apache/hadoop/hive/ql/parse/authorization/TestPrivilegesV1.java
fd827ad
ql/src/test/org/apache/hadoop/hive/ql/parse/authorization/TestPrivilegesV2.java
9499986
ql/src/test/queries/clientpositive/alter_rename_table.q PRE-CREATION
ql/src/test/results/clientnegative/alter_concatenate_indexed_table.q.out
500d45d
ql/src/test/results/clientnegative/alter_view_failure6.q.out cfbaca8
ql/src/test/results/clientnegative/merge_negative_1.q.out 95f6678
ql/src/test/results/clientnegative/merge_negative_2.q.out b3422e1
ql/src/test/results/clientnegative/show_columns3.q.out 09068b7
ql/src/test/results/clientnegative/show_tableproperties1.q.out ca54088
ql/src/test/results/clientnegative/temp_table_index.q.out 8ec5c0a
ql/src/test/results/clientpositive/alter_rename_table.q.out PRE-CREATION
ql/src/test/results/clientpositive/drop_multi_partitions.q.out eae57f3
ql/src/test/results/clientpositive/input3.q.out 547449c
ql/src/test/results/clientpositive/insert2_overwrite_partitions.q.out 21bd257
ql/src/test/results/clientpositive/show_create_table_db_table.q.out 0119471
ql/src/test/results/clientpositive/show_tblproperties.q.out 80db5e4
ql/src/test/results/clientpositive/temp_table_names.q.out 940684c
ql/src/test/results/clientpositive/temp_table_precedence.q.out 1075b2c
Diff: https://reviews.apache.org/r/23674/diff/
Testing
-------
Thanks,
Navis Ryu
