----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68140/ -----------------------------------------------------------
(Updated Aug. 14, 2018, 9:22 a.m.) Review request for oozie, András Piros and Kinga Marton. Changes ------- Refactored the code, to keep filter field listing in a single class (SLASummaryGetForFilterJPAExecutor). 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/executor/jpa/sla/SLASummaryGetForFilterJPAExecutor.java b54161e9 core/src/main/java/org/apache/oozie/executor/jpa/sla/SelectQuery.java PRE-CREATION core/src/main/java/org/apache/oozie/servlet/V2SLAServlet.java 3982d1e0 core/src/test/java/org/apache/oozie/executor/jpa/sla/TestSLASummaryGetForFilterJPAExecutor.java PRE-CREATION core/src/test/java/org/apache/oozie/executor/jpa/sla/TestSelectQuery.java PRE-CREATION core/src/test/java/org/apache/oozie/servlet/TestV2SLAServlet.java aa633225 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/3/ Changes: https://reviews.apache.org/r/68140/diff/2-3/ Testing ------- Thanks, Andras Salamon
