----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67129/#review203110 -----------------------------------------------------------
core/src/test/java/org/apache/oozie/service/TestJMSAccessorService.java Lines 160 (patched) <https://reviews.apache.org/r/67129/#comment285187> Wouldn't use lambdas, an anonymous class is better IMO. core/src/test/java/org/apache/oozie/service/TestJMSAccessorService.java Line 226 (original), 207 (patched) <https://reviews.apache.org/r/67129/#comment285186> Wouldn't use lambdas, an anonymous class is better IMO. core/src/test/java/org/apache/oozie/service/TestJMSAccessorService.java Line 265 (original), 252 (patched) <https://reviews.apache.org/r/67129/#comment285185> Wouldn't use lambdas, an anonymous class is better IMO. core/src/test/java/org/apache/oozie/test/XTestCase.java Lines 806-813 (patched) <https://reviews.apache.org/r/67129/#comment285188> Nice :) - András Piros On May 15, 2018, 12:43 p.m., Peter Bacsko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67129/ > ----------------------------------------------------------- > > (Updated May 15, 2018, 12:43 p.m.) > > > Review request for oozie, András Piros and Peter Cseh. > > > Repository: oozie-git > > > Description > ------- > > Proposed fir for OOZIE-3240. > > Note that most likely this patch will also solve OOZIE-3239 (or other tests > that fail in this class). > > > Diffs > ----- > > core/src/test/java/org/apache/oozie/service/TestJMSAccessorService.java > 41241d2533bf6c3f32f0fca5d76ad5b960bff1a1 > core/src/test/java/org/apache/oozie/test/XTestCase.java > f471b889896e9a37a59c3b8ad1c47633eed3e0c0 > > > Diff: https://reviews.apache.org/r/67129/diff/1/ > > > Testing > ------- > > > Thanks, > > Peter Bacsko > >
