----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56158/#review163796 -----------------------------------------------------------
core/src/main/java/org/apache/oozie/command/coord/CoordCommandUtils.java (line 384) <https://reviews.apache.org/r/56158/#comment235302> End of functions calculation should not include initial instance calculation. It's purely on nominal time. Endof functions at dataset frequency and at datasets instance should return the same value. - Purshotam Shah On Feb. 1, 2017, 8:44 a.m., Satish Saley wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56158/ > ----------------------------------------------------------- > > (Updated Feb. 1, 2017, 8:44 a.m.) > > > Review request for oozie. > > > Bugs: OOZIE-2630 > https://issues.apache.org/jira/browse/OOZIE-2630 > > > Repository: oozie-git > > > Description > ------- > > [OOZIE-2630] Amend patch for OOZIE-2630 > > > Diffs > ----- > > core/src/main/java/org/apache/oozie/command/coord/CoordCommandUtils.java > 3a7a930 > core/src/main/java/org/apache/oozie/coord/CoordELFunctions.java bc18f4d > > core/src/test/java/org/apache/oozie/command/coord/TestCoordCommandUtils.java > 22d1f61 > core/src/test/resources/coord-dataset-endOfDays.xml 6aa3f00 > core/src/test/resources/coord-dataset-endOfMonths.xml 0ea5cac > core/src/test/resources/coord-dataset-endOfWeeks.xml 7879bf4 > > Diff: https://reviews.apache.org/r/56158/diff/ > > > Testing > ------- > > Tested locally > > > Thanks, > > Satish Saley > >