Re: Review Request 54746: SQOOP-3083 Fault injection targets no longer work after SQOOP-2983 due to introducing try with resource

2016-12-16 Thread Attila Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54746/#review159428 --- Ship it! Hey Anna, Thanks for spotting+patching this!

Re: Review Request 54746: SQOOP-3083 Fault injection targets no longer work after SQOOP-2983 due to introducing try with resource

2016-12-15 Thread Anna Szonyi
> On Dec. 15, 2016, 4:32 p.m., Szabolcs Vasas wrote: > > src/test/aop/build/aop.xml, line 41 > > > > > > It seems this patch contains the content of SQOOP-3084 too, so if won't > > apply if SQOOP-3084 gets committed

Re: Review Request 54746: SQOOP-3083 Fault injection targets no longer work after SQOOP-2983 due to introducing try with resource

2016-12-15 Thread Anna Szonyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54746/ --- (Updated Dec. 15, 2016, 7:37 p.m.) Review request for Sqoop, Boglarka Egyed,

Re: Review Request 54746: SQOOP-3083 Fault injection targets no longer work after SQOOP-2983 due to introducing try with resource

2016-12-15 Thread Szabolcs Vasas
--- 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)

Re: Review Request 54746: SQOOP-3083 Fault injection targets no longer work after SQOOP-2983 due to introducing try with resource

2016-12-15 Thread Erzsebet Szilagyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54746/#review159312 --- Ship it! Ship It! - Erzsebet Szilagyi On Dec. 14, 2016,

Re: Review Request 54746: SQOOP-3083 Fault injection targets no longer work after SQOOP-2983 due to introducing try with resource

2016-12-14 Thread Anna Szonyi
--- 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,

Review Request 54746: SQOOP-3083 Fault injection targets no longer work after SQOOP-2983 due to introducing try with resource

2016-12-14 Thread Anna Szonyi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54746/ --- Review request for Sqoop, Boglarka Egyed, Attila Szabo, Szabolcs Vasas, and