----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67129/#review203109 -----------------------------------------------------------
core/src/test/java/org/apache/oozie/service/TestJMSAccessorService.java Line 86 (original) <https://reviews.apache.org/r/67129/#comment285182> I removed these catch blocks, they didn't serve any purpose whatsoever. core/src/test/java/org/apache/oozie/service/TestJMSAccessorService.java Lines 142-146 (original), 127 (patched) <https://reviews.apache.org/r/67129/#comment285183> This was absolutely unnecessary here. core/src/test/java/org/apache/oozie/test/XTestCase.java Lines 806 (patched) <https://reviews.apache.org/r/67129/#comment285184> Small enhancement for JDK8. - Peter Bacsko On máj. 15, 2018, 12:43 du, Peter Bacsko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67129/ > ----------------------------------------------------------- > > (Updated máj. 15, 2018, 12:43 du) > > > 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 > >
