hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Harish Butani" <rhbut...@gmail.com>
Subject Review Request 20738: HIVE-2584: Alter table should accept database name
Date Sat, 26 Apr 2014 00:31:26 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20738/
-----------------------------------------------------------

Review request for hive.


Bugs: HIVE-2584
    https://issues.apache.org/jira/browse/HIVE-2584


Repository: hive-git


Description
-------

It would be nice if alter table accepts database name.
For example:
This would be more useful in certain usecases:
alter table DB.Tbl set location <location>;
rather than 2 statements.
use DB;
alter table Tbl set location <location>;


Diffs
-----

  metastore/src/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java 8345d70 
  ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java ec68e7c 
  ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 947b65c 
  ql/src/java/org/apache/hadoop/hive/ql/parse/BaseSemanticAnalyzer.java 448dae2 
  ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 0f60fcb 
  ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g b34f53b 
  ql/src/test/queries/clientnegative/alter_table_rename.q PRE-CREATION 
  ql/src/test/queries/clientpositive/alter6.q PRE-CREATION 
  ql/src/test/results/clientnegative/alter_partition_coltype_2columns.q.out e1f9a27 
  ql/src/test/results/clientnegative/alter_table_rename.q.out PRE-CREATION 
  ql/src/test/results/clientnegative/archive_partspec3.q.out c85e9a2 
  ql/src/test/results/clientpositive/alter6.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/drop_multi_partitions.q.out 31cd197 
  ql/src/test/results/clientpositive/input3.q.out 58231b1 
  ql/src/test/results/clientpositive/insert2_overwrite_partitions.q.out fcc551e 
  ql/src/test/results/clientpositive/show_create_table_db_table.q.out d36e8b0 

Diff: https://reviews.apache.org/r/20738/diff/


Testing
-------

  


Thanks,

Harish Butani


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