----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9739/#review18092 -----------------------------------------------------------
I realized I forgot to give one comment earlier. Related to 1189 but can be done as part of this patch In OozieCLI, createSLAoptions(), { Option filter = new Option(FILTER_OPTION, true, "filter of SLA events"); } Specify what filter options are available like slaid, appname - Virag Kothari On March 4, 2013, 11:19 p.m., Ryota Egashira wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9739/ > ----------------------------------------------------------- > > (Updated March 4, 2013, 11:19 p.m.) > > > Review request for oozie. > > > Description > ------- > > https://issues.apache.org/jira/browse/OOZIE-1246 > > > This addresses bug OOZIE_1246. > https://issues.apache.org/jira/browse/OOZIE_1246 > > > Diffs > ----- > > trunk/core/src/main/java/org/apache/oozie/SLAEventBean.java 1452464 > trunk/core/src/main/java/org/apache/oozie/util/db/SLADbOperations.java > 1452464 > > Diff: https://reviews.apache.org/r/9739/diff/ > > > Testing > ------- > > tested locally > > > Thanks, > > Ryota Egashira > >