----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54746/#review159313 -----------------------------------------------------------
src/test/aop/build/aop.xml (line 41) <https://reviews.apache.org/r/54746/#comment230288> It seems this patch contains the content of SQOOP-3084 too, so if won't apply if SQOOP-3084 gets committed first. Or do you want to merge to 2 changes? - Szabolcs Vasas On Dec. 14, 2016, 3:45 p.m., Anna Szonyi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54746/ > ----------------------------------------------------------- > > (Updated Dec. 14, 2016, 3:45 p.m.) > > > Review request for Sqoop, Boglarka Egyed, Attila Szabo, Szabolcs Vasas, and > Erzsebet Szilagyi. > > > Bugs: SQOOP-3083 > https://issues.apache.org/jira/browse/SQOOP-3083 > > > Repository: sqoop-trunk > > > Description > ------- > > SQOOP-3083 Fault injection targets no longer work after SQOOP-2983 due to > introducing try with resource > > > Diffs > ----- > > ivy/libraries.properties e2c2516 > src/test/aop/build/aop.xml 77320aa > > Diff: https://reviews.apache.org/r/54746/diff/ > > > Testing > ------- > > ant compile, ant test, ant jar-fault-inject > > > Thanks, > > Anna Szonyi > >