-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/9626/
-----------------------------------------------------------

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

Reply via email to