----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3752/#review8360 -----------------------------------------------------------
Shwetha, any update on this JIRA? When uploading a new patch, please attach it directly in the JIRA and then close this review request as discarded, thx. - Alejandro Abdelnur On March 7, 2012, 9:43 a.m., shwethags wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3752/ > ----------------------------------------------------------- > > (Updated March 7, 2012, 9:43 a.m.) > > > Review request for oozie. > > > Description > ------- > > I have an EL extension today(0,0) which maps to start day of nominal time. > This is used to specify startInstance, endInstance and instance in dataIn and > dataOut of coordinator. > > In CoordCommandUtils.resolveInstanceRange(), getInstanceNumber has to return > the instance number with respect to current. So, for coord-action-create-inst > context, I have mapped today to current and hence getInstanceNumber returns > the correct number. But later in resolveInstanceRange(), getFuncType is > called with startInstance value which is today in this case and it maps to > UNEXPECTED and throws up. getFuncType should be passed the evaluation of > coord-action-create-inst context > > > This addresses bug OOZIE-674. > https://issues.apache.org/jira/browse/OOZIE-674 > > > Diffs > ----- > > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordCommandUtils.java > 1297889 > trunk/core/src/main/java/org/apache/oozie/coord/CoordELFunctions.java > 1297889 > > trunk/core/src/test/java/org/apache/oozie/command/coord/CoordELExtension.java > PRE-CREATION > > trunk/core/src/test/java/org/apache/oozie/command/coord/TestCoordELExtensions.java > PRE-CREATION > trunk/core/src/test/resources/oozie-site-coordel.xml PRE-CREATION > > Diff: https://reviews.apache.org/r/3752/diff/ > > > Testing > ------- > > UT - TestCoordActionMaterializeCommand > Tested with coord:current instance range and EL extension > > > Thanks, > > shwethags > >
