----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37673/#review96088 -----------------------------------------------------------
Java 7 or newer only support try-with-resources. Requirements say Java 6 should be supported for development: http://sqoop.apache.org/docs/1.99.6/DevEnv.html#java. - Abraham Elmahrek On Aug. 21, 2015, 7:59 a.m., Colin Ma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37673/ > ----------------------------------------------------------- > > (Updated Aug. 21, 2015, 7:59 a.m.) > > > Review request for Sqoop. > > > Repository: sqoop-sqoop2 > > > Description > ------- > > Fix resource leak problem in CommonRepositoryHandler > > > Diffs > ----- > > > repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryHandler.java > c090d8e > > Diff: https://reviews.apache.org/r/37673/diff/ > > > Testing > ------- > > > Thanks, > > Colin Ma > >
