----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1098/#review1101 -----------------------------------------------------------
trunk/cli/src/java/org/apache/hadoop/hive/cli/CliDriver.java <https://reviews.apache.org/r/1098/#comment2233> Rename this to rc to avoid confusion with the processReader method name. trunk/contrib/src/java/org/apache/hadoop/hive/contrib/util/typedbytes/TypedBytesWritableInput.java <https://reviews.apache.org/r/1098/#comment2234> Fix indentation for the try block. - John On 2011-07-13 14:28:47, chinna wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/1098/ > ----------------------------------------------------------- > > (Updated 2011-07-13 14:28:47) > > > Review request for hive and John Sichi. > > > Summary > ------- > > There are couple of resource leaks. > For example, > In CliDriver.java, Method :- processReader() the buffered reader is not > closed. > > Also there are risk(s) of resource(s) getting leaked , in such cases > refactored the code to move closing of resources in finally block. > > > This addresses bug HIVE-1884. > https://issues.apache.org/jira/browse/HIVE-1884 > > > Diffs > ----- > > trunk/cli/src/java/org/apache/hadoop/hive/cli/CliDriver.java 1145950 > > trunk/contrib/src/java/org/apache/hadoop/hive/contrib/util/typedbytes/TypedBytesWritableInput.java > 1145950 > trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 1145950 > trunk/ql/src/java/org/apache/hadoop/hive/ql/io/RCFileInputFormat.java > 1145950 > > Diff: https://reviews.apache.org/r/1098/diff > > > Testing > ------- > > All unit tests passed > > > Thanks, > > chinna > >