Re: Review Request: Resource Leak: Fix the File handle leak in EximUtil.java

2012-07-10 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5777/#review9021 --- Ship it! Ship It! http://svn.apache.org/repos/asf/hive/trunk/ql/src

Re: Review Request: Resource Leak: Fix the File handle leak in EximUtil.java

2012-07-05 Thread Kanna Karanam
> On July 5, 2012, 8:12 p.m., Carl Steinbach wrote: > > Thanks Carl. Please find the comments below. I tested these changes on Unix as well as on Windows and exim unit tests are passing. > On July 5, 2012, 8:12 p.m., Carl Steinbach wrote: > > http://svn.apache.org/repos/asf/hive/trunk/ql/src/

Re: Review Request: Resource Leak: Fix the File handle leak in EximUtil.java

2012-07-05 Thread Carl Steinbach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5777/#review8890 --- http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/had

Review Request: Resource Leak: Fix the File handle leak in EximUtil.java

2012-07-05 Thread Kanna Karanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5777/ --- Review request for hive, Carl Steinbach, Edward Capriolo, and Ashutosh Chauhan.