----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11659/#review21592 -----------------------------------------------------------
http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/executor/jpa/sla/SLASummaryGetForFilterJPAExecutor.java <https://reviews.apache.org/r/11659/#comment44649> remove the block, firstCondition will be always true http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/servlet/V2SLAServlet.java <https://reviews.apache.org/r/11659/#comment44648> pass instrumentation name - Virag Kothari On June 5, 2013, 6:36 p.m., Rohini Palaniswamy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11659/ > ----------------------------------------------------------- > > (Updated June 5, 2013, 6:36 p.m.) > > > Review request for oozie. > > > Description > ------- > > Rest API to fetch sla summary for the UI. Did not add to Web Services > documentation as previous sla API was not documented. Will expose in > documentation once we stabilize. > > > This addresses bug OOZIE-1400. > https://issues.apache.org/jira/browse/OOZIE-1400 > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/oozie/trunk/client/src/main/java/org/apache/oozie/cli/OozieCLI.java > 1489927 > > http://svn.apache.org/repos/asf/oozie/trunk/client/src/main/java/org/apache/oozie/client/OozieClient.java > 1489927 > > http://svn.apache.org/repos/asf/oozie/trunk/client/src/main/java/org/apache/oozie/client/rest/JsonTags.java > 1489927 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/executor/jpa/sla/SLASummaryGetForFilterJPAExecutor.java > PRE-CREATION > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/servlet/SLAServlet.java > 1489927 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/servlet/V2SLAServlet.java > PRE-CREATION > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/main/java/org/apache/oozie/sla/SLASummaryBean.java > 1489927 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/client/TestWorkflowClient.java > 1489927 > > http://svn.apache.org/repos/asf/oozie/trunk/core/src/test/java/org/apache/oozie/servlet/TestV2SLAServlet.java > PRE-CREATION > > http://svn.apache.org/repos/asf/oozie/trunk/webapp/src/main/webapp/WEB-INF/web.xml > 1489927 > > Diff: https://reviews.apache.org/r/11659/diff/ > > > Testing > ------- > > Unit tests added > > > Thanks, > > Rohini Palaniswamy > >
