-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/66359/
-----------------------------------------------------------
Review request for hive, Peter Vary and Adam Szita.
Bugs: HIVE-19076
https://issues.apache.org/jira/browse/HIVE-19076
Repository: hive-git
Description
-------
The TestFunctions tests revealed that NPE is thrown in some cases. These NPEs
could be prevented with a simple null check and a MetaException with a proper
error message should be thrown instead.
Also there are some alter function tests where InvalidObjectException is thrown
with Embedded MetaStore, but TApplicationException it thrown with Remote
MetaStore. The reason is that the InvalidObjectException is not defined for the
alter_function method in the thrift interface, so we got the
TApplicationException when the InvalidObjectException was thrown. In these
cases the InvalidObjectException could be handled on the server side and
re-throw it as a MetaException
Diffs
-----
standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
8a5de09
standalone-metastore/src/test/java/org/apache/hadoop/hive/metastore/client/TestFunctions.java
d504f34
Diff: https://reviews.apache.org/r/66359/diff/1/
Testing
-------
Thanks,
Marta Kuczora
