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



trunk/client/src/main/java/org/apache/oozie/cli/OozieCLI.java
<https://reviews.apache.org/r/2274/#comment5576>

    This change is from previous commit



trunk/core/src/main/java/org/apache/oozie/util/CoordActionsInDateRange.java
<https://reviews.apache.org/r/2274/#comment5577>

    Documentation comment for private method
    Its better to use implementation comment (/*...*/)
     


- Virag


On 2011-10-08 02:39:35, Mona Chitnis wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/2274/
> -----------------------------------------------------------
> 
> (Updated 2011-10-08 02:39:35)
> 
> 
> 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 1180295 
>   trunk/client/src/main/java/org/apache/oozie/client/rest/RestConstants.java 
> 1180295 
>   trunk/core/src/main/java/org/apache/oozie/CoordinatorEngine.java 1180295 
>   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 
> 
> Diff: https://reviews.apache.org/r/2274/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Mona
> 
>

Reply via email to