-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23674/
-----------------------------------------------------------
Review request for hive.
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
-----
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
c09f4a4
metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java
5a56ced
metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java 5e2cad7
ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java ee074ea
ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java 4450ad3
ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java a891838
ql/src/java/org/apache/hadoop/hive/ql/optimizer/IndexUtils.java ae87aac
ql/src/java/org/apache/hadoop/hive/ql/optimizer/index/RewriteGBUsingIndex.java
11a6d07
ql/src/java/org/apache/hadoop/hive/ql/parse/BaseSemanticAnalyzer.java 22945e3
ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java c0322fb
ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g f5d0602
ql/src/java/org/apache/hadoop/hive/ql/parse/IndexUpdater.java 856ec2f
ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java d38270c
ql/src/java/org/apache/hadoop/hive/ql/parse/authorization/HiveAuthorizationTaskFactoryImpl.java
c32d81e
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/test/org/apache/hadoop/hive/ql/metadata/TestHive.java d218271
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/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/drop_multi_partitions.q.out eae57f3
ql/src/test/results/clientpositive/insert2_overwrite_partitions.q.out 21bd257
ql/src/test/results/clientpositive/show_columns.q.out 1b74802
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
