----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67885/#review207318 -----------------------------------------------------------
core/src/main/java/org/apache/oozie/service/AsyncXCommandExecutor.java Lines 69 (patched) <https://reviews.apache.org/r/67885/#comment290675> Todo: probably this should be configurable (add property) core/src/main/java/org/apache/oozie/service/AsyncXCommandExecutor.java Lines 127 (patched) <https://reviews.apache.org/r/67885/#comment290676> Extract constants core/src/main/java/org/apache/oozie/service/CallableQueueService.java Lines 473 (patched) <https://reviews.apache.org/r/67885/#comment290677> Constant - Peter Bacsko On aug. 15, 2018, 9:49 de, Peter Bacsko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67885/ > ----------------------------------------------------------- > > (Updated aug. 15, 2018, 9:49 de) > > > Review request for oozie, András Piros, Peter Cseh, Kinga Marton, and Robert > Kanter. > > > Repository: oozie-git > > > Description > ------- > > Still just a POC. > > Tests are almost completely missing. > > > 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/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/6/ > > > Testing > ------- > > Executed TestCallableQueueService which passed completely. > > > Thanks, > > Peter Bacsko > >