----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9626/#review17418 -----------------------------------------------------------
Ship it! Sounds reasonable. - Mona Chitnis On Feb. 26, 2013, 8:06 p.m., Rohini Palaniswamy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9626/ > ----------------------------------------------------------- > > (Updated Feb. 26, 2013, 8:06 p.m.) > > > Review request for oozie. > > > Description > ------- > > Use a currentRange function to move common code out of the loop and avoid CPU > cycles spent in initial instance and current(0) calculation. > > > This addresses bug OOZIE-1207. > https://issues.apache.org/jira/browse/OOZIE-1207 > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/command/coord/CoordCommandUtils.java > 1450366 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/coord/CoordELFunctions.java > 1450366 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/util/ParamChecker.java > 1450366 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/resources/oozie-default.xml > 1450366 > > Diff: https://reviews.apache.org/r/9626/diff/ > > > Testing > ------- > > Current unit tests in CoordActionInputCheckXCommand cover the functionality. > No new unit tests added. > > > Thanks, > > Rohini Palaniswamy > >
