-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1978/
-----------------------------------------------------------
(Updated 2011-10-21 17:57:45.867679)
Review request for hive, Carl Steinbach and John Sichi.
Changes
-------
Now i have updated the patch like "throw new HiveException(e.toString())" these
are changed to "throw new HiveException(e)" now it is capturing the exception
stack trace in *.q.out and log files also
Summary
-------
Fixed Log related Check style Comments like including root exception and adding
messages.
This addresses bug HIVE-2178.
https://issues.apache.org/jira/browse/HIVE-2178
Diffs (updated)
-----
trunk/jdbc/src/java/org/apache/hadoop/hive/jdbc/HiveConnection.java 1187471
trunk/jdbc/src/java/org/apache/hadoop/hive/jdbc/HiveDataSource.java 1187471
trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 1187471
trunk/ql/src/java/org/apache/hadoop/hive/ql/metadata/Partition.java 1187471
trunk/serde/src/java/org/apache/hadoop/hive/serde2/dynamic_type/SimpleCharStream.java
1187471
trunk/serde/src/java/org/apache/hadoop/hive/serde2/thrift/TBinarySortableProtocol.java
1187471
Diff: https://reviews.apache.org/r/1978/diff
Testing
-------
Ran all unit tests
Thanks,
chinna
