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

(Updated Oct. 28, 2013, 10:30 p.m.)


Review request for hive.


Changes
-------

Adding the exception to the log.  Verified by checking hive.log.

In my observation, the NPE for non-partitioned table use-case occurred at 
DDLSemanticAnalyzer 2951 : if (parts.isempty()), as parts is null after 
swallowing the exception.


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


Repository: hive-git


Description
-------

Throwing the offending exception, that was swallowed.  Also changing the 
underlying exception to use the standard error message.


Diffs (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java b0f124b 
  ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 037191a 
  ql/src/test/results/clientnegative/alter_rename_partition_failure3.q.out 
384fcbe 
  ql/src/test/results/clientnegative/touch2.q.out 045121a 

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


Testing
-------


Thanks,

Szehon Ho

Reply via email to