----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68140/ -----------------------------------------------------------
(Updated Sept. 4, 2018, 8:36 a.m.) Review request for oozie, András Piros and Kinga Marton. Changes ------- I also separated TestV2SLAServlet into multiple classes: TestV2SLAServlet, TestV2SLAServletBundle, TestV2SLAServletRun 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 (updated) ----- client/src/main/java/org/apache/oozie/client/OozieClient.java 949b4532 core/src/main/java/org/apache/oozie/FilterParser.java PRE-CREATION core/src/main/java/org/apache/oozie/executor/jpa/sla/SLASummaryGetForFilterJPAExecutor.java b54161e9 core/src/main/java/org/apache/oozie/servlet/V2SLAServlet.java 3982d1e0 core/src/test/java/org/apache/oozie/TestFilterParser.java PRE-CREATION core/src/test/java/org/apache/oozie/executor/jpa/sla/TestSLASummaryGetForFilterJPAExecutor.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 aa633225 core/src/test/java/org/apache/oozie/servlet/TestV2SLAServletBundle.java PRE-CREATION core/src/test/java/org/apache/oozie/servlet/TestV2SLAServletRun.java PRE-CREATION core/src/test/java/org/apache/oozie/servlet/V2SLAServletTestCase.java PRE-CREATION docs/src/site/twiki/DG_SLAMonitoring.twiki c91c227d webapp/src/main/webapp/console/sla/css/oozie-sla.css d2f2deee webapp/src/main/webapp/console/sla/js/oozie-sla.js 2ecad228 webapp/src/main/webapp/console/sla/oozie-sla.html e5bf6275 Diff: https://reviews.apache.org/r/68140/diff/5/ Changes: https://reviews.apache.org/r/68140/diff/4-5/ Testing ------- Thanks, Andras Salamon
