-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4222/
-----------------------------------------------------------
(Updated 2012-03-09 11:06:47.528914)
Review request for oozie.
Changes
-------
New patch uploaded after incorporating review comments. Thanks for taking time
to review.
Summary
-------
checkMultipleTimeInstances doesn't work for EL extensions. EX:
${coord:formatTime(coord:current(0),'yyyy-MM-dd')}
This addresses bug OOZIE-675.
https://issues.apache.org/jira/browse/OOZIE-675
Diffs (updated)
-----
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordSubmitXCommand.java
1297367
trunk/core/src/main/java/org/apache/oozie/util/ELEvaluator.java 1297367
trunk/core/src/test/java/org/apache/oozie/command/coord/TestCoordSubmitXCommand.java
1297367
trunk/core/src/test/java/org/apache/oozie/util/TestELEvaluator.java 1297367
trunk/core/src/test/resources/coord-multiple-input-instance4.xml PRE-CREATION
trunk/core/src/test/resources/coord-multiple-output-instance4.xml
PRE-CREATION
Diff: https://reviews.apache.org/r/4222/diff
Testing
-------
* New test case added to TestCoordSubmitXCommand.java for verifying the changes
* Added additional test cases to TestELEvaluator.java as well
Thanks,
Srikanth
