----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37567/#review95894 -----------------------------------------------------------
Ship it! Ship It! - Jarek Cecho On Aug. 19, 2015, 5:14 a.m., Colin Ma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37567/ > ----------------------------------------------------------- > > (Updated Aug. 19, 2015, 5:14 a.m.) > > > Review request for Sqoop. > > > Repository: sqoop-sqoop2 > > > Description > ------- > > Fix the resource leak problem in ProviderAsserts and DatabaseProvider > > > Diffs > ----- > > > common-test/src/main/java/org/apache/sqoop/common/test/asserts/ProviderAsserts.java > 4d6c8ec > > common-test/src/main/java/org/apache/sqoop/common/test/db/DatabaseProvider.java > dd4e546 > > Diff: https://reviews.apache.org/r/37567/diff/ > > > Testing > ------- > > > Thanks, > > Colin Ma > >
