----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68140/#review208795 -----------------------------------------------------------
Ship it! Ship It! - András Piros On Sept. 19, 2018, 4:07 p.m., Andras Salamon wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68140/ > ----------------------------------------------------------- > > (Updated Sept. 19, 2018, 4:07 p.m.) > > > Review request for oozie, András Piros and Kinga Marton. > > > Repository: oozie-git > > > Description > ------- > > Added lots of new filter fields, also refactored > SLASummaryGetForFilterJPAExecutor class to eliminate FindBugs errors. I'm not > sure about the filter field names. > > > Diffs > ----- > > client/src/main/java/org/apache/oozie/client/OozieClient.java 949b4532f > client/src/main/java/org/apache/oozie/client/rest/RestConstants.java > 9873ff3ad > core/src/main/java/org/apache/oozie/FilterParser.java PRE-CREATION > > core/src/main/java/org/apache/oozie/executor/jpa/sla/SLASummaryGetForFilterJPAExecutor.java > b54161e98 > core/src/main/java/org/apache/oozie/servlet/V2SLAServlet.java 3982d1e06 > core/src/test/java/org/apache/oozie/TestFilterParser.java PRE-CREATION > > core/src/test/java/org/apache/oozie/executor/jpa/sla/TestSLASummaryGetForFilterJPAExecutorFilterCollection.java > PRE-CREATION > core/src/test/java/org/apache/oozie/servlet/TestV2SLAServlet.java aa633225b > core/src/test/java/org/apache/oozie/servlet/TestV2SLAServletBundle.java > PRE-CREATION > > core/src/test/java/org/apache/oozie/servlet/TestV2SLAServletIntegration.java > PRE-CREATION > > core/src/test/java/org/apache/oozie/servlet/TestV2SLAServletSLAJSONResponse.java > PRE-CREATION > core/src/test/java/org/apache/oozie/servlet/V2SLAServletTestCase.java > PRE-CREATION > docs/src/site/markdown/DG_SLAMonitoring.md 0831b93bd > webapp/src/main/webapp/console/sla/css/oozie-sla.css d2f2deeec > webapp/src/main/webapp/console/sla/js/oozie-sla.js 2ecad228a > webapp/src/main/webapp/console/sla/oozie-sla.html e5bf6275a > > > Diff: https://reviews.apache.org/r/68140/diff/7/ > > > Testing > ------- > > > Thanks, > > Andras Salamon > >