hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Navis Ryu" <navis....@nexr.com>
Subject Review Request 23674: Handle db qualified names consistently across all HiveQL statements
Date Fri, 18 Jul 2014 04:00:26 GMT

-----------------------------------------------------------
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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message