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

(Updated March 20, 2015, 12:08 a.m.)


Review request for hive, Ashutosh Chauhan, Szehon Ho, and Xuefu Zhang.


Changes
-------

Fixed three failed tests and add a new case to compute column stats for a table 
not under default db.


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


Repository: hive-git


Description
-------

Currently Currently "analyze table compute statistics for columns" command can 
not compute column stats for a table in a different database since it does not 
support qualified table name. You need switch to that table database in order 
to compute its column stats. So does it for ALTER TABLE .. UPDATE STATISTICS 
FOR COLUMN command.
This JIRA will provide the support to qualified table name in analyze and 
update column stats commands.


Diffs (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/exec/ColumnStatsTask.java 
e2f696eea7b472d4e417629eedd33406b41caf98 
  ql/src/java/org/apache/hadoop/hive/ql/exec/ColumnStatsUpdateTask.java 
28c17b75356eb4ffaac2193a80b27bdd67d98009 
  ql/src/java/org/apache/hadoop/hive/ql/parse/ColumnStatsSemanticAnalyzer.java 
c83523e507f1003eff1a821c216a7f6904f293ed 
  ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 
b9e15a1b2f60b291759444a4407acaf1e0d9c515 
  ql/src/test/queries/clientpositive/alter_partition_update_status.q 
1eee9a506f3854eace7deaa3f4ec02c1f7bd3479 
  ql/src/test/queries/clientpositive/alter_table_update_status.q 
fd45cd426c17a7bfa1371081dd81ff6eb1b7353d 
  ql/src/test/queries/clientpositive/columnstats_partlvl.q 
82a9e0f0fef0c9bf909028f23ccd47ee07a61a2b 
  ql/src/test/queries/clientpositive/columnstats_tbllvl.q 
07cc959ec6bb9f5e1f1ba3184da5ce3fc8bf951b 
  ql/src/test/results/clientpositive/alter_partition_update_status.q.out 
7e33a7e8ecc07b2b9d8ab8b0f12b383e5a442cf3 
  ql/src/test/results/clientpositive/alter_table_update_status.q.out 
361359807b9c8c1a2e25a629fd53f026e02957e7 
  ql/src/test/results/clientpositive/columnstats_partlvl.q.out 
a86c5fbd718a490c5973382810cc02c10184925e 
  ql/src/test/results/clientpositive/columnstats_partlvl_dp.q.out 
073b387c7b1d5f174928a110381faa851133c7ea 
  ql/src/test/results/clientpositive/columnstats_tbllvl.q.out 
0ef04f5f4b70d84cd6118c2407d29fc555bac116 
  ql/src/test/results/clientpositive/display_colstats_tbllvl.q.out 
a176d9a2a9db7eb848d1db022c4d99680db1b62c 
  ql/src/test/results/clientpositive/temp_table_display_colstats_tbllvl.q.out 
933d24b6cde934faa013f1ea102b9b753a7d44a4 

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


Testing
-------

1. Performed some manullay tests
2. Added new qtests for those cases
3. Precommit build


Thanks,

Chaoyu Tang

Reply via email to