----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67885/#review208310 -----------------------------------------------------------
core/src/main/java/org/apache/oozie/service/AsyncXCommandExecutor.java Lines 1 (patched) <https://reviews.apache.org/r/67885/#comment292140> Apache 2.0 license header missing. core/src/main/java/org/apache/oozie/service/CallableQueueService.java Lines 562 (patched) <https://reviews.apache.org/r/67885/#comment292142> Typo: `by looking at the` core/src/test/java/org/apache/oozie/service/TestAsyncXCommandExecutor.java Lines 75 (patched) <https://reviews.apache.org/r/67885/#comment292143> Remove newline - András Piros On Sept. 4, 2018, 12:05 p.m., Peter Bacsko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67885/ > ----------------------------------------------------------- > > (Updated Sept. 4, 2018, 12:05 p.m.) > > > Review request for oozie, András Piros, Peter Cseh, Kinga Marton, and Robert > Kanter. > > > Repository: oozie-git > > > Description > ------- > > See upstream JIRA for details > > > Diffs > ----- > > core/src/main/java/org/apache/oozie/service/AsyncXCommandExecutor.java > PRE-CREATION > core/src/main/java/org/apache/oozie/service/CallableQueueService.java > ef8d58da5 > core/src/main/java/org/apache/oozie/util/PriorityDelayQueue.java 75c20698c > core/src/main/resources/oozie-default.xml b69d2c9aa > core/src/test/java/org/apache/oozie/service/TestAsyncXCommandExecutor.java > PRE-CREATION > core/src/test/java/org/apache/oozie/service/TestCallableQueueService.java > 9c2a11d6f > > > Diff: https://reviews.apache.org/r/67885/diff/9/ > > > Testing > ------- > > 1. Executed TestCallableQueueService which passed completely. > 2. New unit tests for ASyncXCommandExecutor. > 3. Tried it on a 3-node cluster > > > Thanks, > > Peter Bacsko > >