----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2274/#review2542 -----------------------------------------------------------
trunk/core/src/main/java/org/apache/oozie/util/CoordActionsInDateRange.java <https://reviews.apache.org/r/2274/#comment5710> Not sure why it needs to be static. It is used only once. trunk/core/src/main/java/org/apache/oozie/util/CoordActionsInDateRange.java <https://reviews.apache.org/r/2274/#comment5711> you can move jpa service declaration here. - Angelo K. On 2011-10-10 19:23:54, Mona Chitnis wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2274/ > ----------------------------------------------------------- > > (Updated 2011-10-10 19:23:54) > > > Review request for oozie, Mohammad Islam and Angelo K. Huang. > > > Summary > ------- > > This is on behalf of Kiran N.'s work on oozie logs > > Date range for log retrieval > - User can add the command line option -date(Ex.: -date > 2011-01-01T01:00Z::2011-05-31T23:59Z) along with the -log option > > > Diffs > ----- > > trunk/client/src/main/java/org/apache/oozie/cli/OozieCLI.java 1181106 > trunk/client/src/main/java/org/apache/oozie/client/rest/RestConstants.java > 1181106 > trunk/core/src/main/java/org/apache/oozie/CoordinatorEngine.java 1181106 > trunk/core/src/main/java/org/apache/oozie/util/CoordActionsInDateRange.java > PRE-CREATION > > trunk/core/src/test/java/org/apache/oozie/util/TestCoordActionsInDateRange.java > PRE-CREATION > trunk/release-log.txt 1181106 > > Diff: https://reviews.apache.org/r/2274/diff > > > Testing > ------- > > > Thanks, > > Mona > >
