> On 2012-05-15 18:28:20, Virag Kothari wrote: > > trunk/core/src/test/java/org/apache/oozie/util/TestLogStreamer.java, line > > 271 > > <https://reviews.apache.org/r/4881/diff/1/?file=104416#file104416line271> > > > > did u mean 2012-03-24 > > Mona Chitnis wrote: > No, instead the parameter passed should be 2012, 4, 23. Fixing the date > values in the log lines too
adding short comment for this - month 00=Jan, hence for date 2012-04, calendar object should be provided with month-1 i.e. 3 - Mona ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4881/#review7902 ----------------------------------------------------------- On 2012-04-26 00:15:30, Mona Chitnis wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4881/ > ----------------------------------------------------------- > > (Updated 2012-04-26 00:15:30) > > > Review request for oozie. > > > Summary > ------- > > Oozie.log files can be gzipped by a log setting and then we work with hourly > .gz log files. For a long running job spanning multiple hours, OOZIE-590 > fixed the log retrieval issue from multiple such gz files. I'm adding a unit > test for checking this functionality. > > > This addresses bug OOZIE-821. > https://issues.apache.org/jira/browse/OOZIE-821 > > > Diffs > ----- > > trunk/core/src/test/java/org/apache/oozie/util/TestLogStreamer.java 1326832 > > Diff: https://reviews.apache.org/r/4881/diff > > > Testing > ------- > > > Thanks, > > Mona > >
