Removing code smells
[dmaap/datarouter.git] / datarouter-prov / src / main / java / org / onap / dmaap / datarouter / provisioning / StatisticsServlet.java
old mode 100644 (file)
new mode 100755 (executable)
index 886a136..b069544
@@ -28,7 +28,6 @@ import java.sql.Connection;
 import java.sql.PreparedStatement;\r
 import java.sql.ResultSet;\r
 import java.sql.SQLException;\r
-import java.sql.Statement;\r
 import java.text.ParseException;\r
 import java.text.SimpleDateFormat;\r
 import java.util.Calendar;\r
@@ -44,8 +43,12 @@ import org.onap.dmaap.datarouter.provisioning.beans.EventLogRecord;
 import org.onap.dmaap.datarouter.provisioning.utils.DB;\r
 import org.onap.dmaap.datarouter.provisioning.utils.LOGJSONObject;\r
 \r
+import static org.onap.dmaap.datarouter.provisioning.utils.HttpServletUtils.sendResponseError;\r
+\r
 /**\r
- * This Servlet handles requests to the <Statistics API> and  <Statistics consilidated resultset>,\r
+ * This Servlet handles requests to the <Statistics API> and  <Statistics consilidated\r
+ * resultset>,\r
+ *\r
  * @author Manish Singh\r
  * @version $Id: StatisticsServlet.java,v 1.11 2016/08/10 17:27:02 Manish Exp $\r
  */\r
@@ -54,301 +57,214 @@ import org.onap.dmaap.datarouter.provisioning.utils.LOGJSONObject;
 public class StatisticsServlet extends BaseServlet {\r
 \r
     private static final long TWENTYFOUR_HOURS = (24 * 60 * 60 * 1000L);\r
-    private static final String fmt1 = "yyyy-MM-dd'T'HH:mm:ss'Z'";\r
-    private static final String fmt2 = "yyyy-MM-dd'T'HH:mm:ss.SSS'Z'";\r
+    private static final String FMT1 = "yyyy-MM-dd'T'HH:mm:ss'Z'";\r
+    private static final String FMT2 = "yyyy-MM-dd'T'HH:mm:ss.SSS'Z'";\r
+    public static final String FEEDID = "FEEDID";\r
 \r
 \r
     /**\r
      * DELETE a logging URL -- not supported.\r
      */\r
     @Override\r
-    public void doDelete(HttpServletRequest req, HttpServletResponse resp) throws IOException {\r
+    public void doDelete(HttpServletRequest req, HttpServletResponse resp) {\r
         String message = "DELETE not allowed for the logURL.";\r
         EventLogRecord elr = new EventLogRecord(req);\r
         elr.setMessage(message);\r
         elr.setResult(HttpServletResponse.SC_METHOD_NOT_ALLOWED);\r
-        eventlogger.info(elr);\r
-        resp.sendError(HttpServletResponse.SC_METHOD_NOT_ALLOWED, message);\r
+        eventlogger.error(elr.toString());\r
+        sendResponseError(resp, HttpServletResponse.SC_METHOD_NOT_ALLOWED, message, eventlogger);\r
     }\r
+\r
     /**\r
-     * GET a Statistics URL -- retrieve Statistics data for a feed or subscription.\r
-     * See the <b>Statistics API</b> document for details on how this     method should be invoked.\r
+     * GET a Statistics URL -- retrieve Statistics data for a feed or subscription. See the\r
+     * <b>Statistics API</b> document for details on how this     method should be invoked.\r
      */\r
     @Override\r
-    public void doGet(HttpServletRequest req, HttpServletResponse resp) throws IOException {\r
-\r
+    public void doGet(HttpServletRequest req, HttpServletResponse resp) {\r
         Map<String, String> map = buildMapFromRequest(req);\r
         if (map.get("err") != null) {\r
-            resp.sendError(HttpServletResponse.SC_BAD_REQUEST, "Invalid arguments: "+map.get("err"));\r
+            sendResponseError(resp, HttpServletResponse.SC_BAD_REQUEST, "Invalid arguments: " + map.get("err"), eventlogger);\r
             return;\r
         }\r
         // check Accept: header??\r
-\r
         resp.setStatus(HttpServletResponse.SC_OK);\r
         resp.setContentType(LOGLIST_CONTENT_TYPE);\r
-        ServletOutputStream out = resp.getOutputStream();\r
-\r
-\r
         String outputType = "json";\r
-        String feedids = null;\r
-\r
-        if(req.getParameter("feedid") ==null && req.getParameter("groupid") ==null)\r
-        {\r
-            out.print("Invalid request, Feedid or Group ID is required.");\r
+        if (req.getParameter(FEEDID) == null && req.getParameter(GROUPID) == null) {\r
+            try {\r
+                resp.getOutputStream().print("Invalid request, Feedid or Group ID is required.");\r
+            } catch (IOException ioe) {\r
+                eventlogger.error("PROV0171 StatisticsServlet.doGet: " + ioe.getMessage(), ioe);\r
+            }\r
         }\r
-\r
-        if(req.getParameter("feedid")!=null && req.getParameter("groupid") == null) {\r
-            map.put("feedids", req.getParameter("feedid").replace("|", ",").toString());\r
+        if (req.getParameter(FEEDID) != null && req.getParameter(GROUPID) == null) {\r
+            map.put(FEEDIDS, req.getParameter(FEEDID).replace("|", ","));\r
         }\r
-\r
-        if(req.getParameter("groupid") != null && req.getParameter("feedid") ==null) {\r
-              // String groupid1 = null;\r
-            StringBuffer groupid1 = new  StringBuffer();\r
-\r
-                 try {\r
-                     System.out.println("feeedidsssssssss");\r
-                     groupid1 = this.getFeedIdsByGroupId(Integer.parseInt(req.getParameter("groupid")));\r
-                      System.out.println("feeedids"+req.getParameter("groupid"));\r
-\r
-                        map.put("feedids", groupid1.toString());\r
-                        System.out.println("groupid1" +groupid1.toString());\r
-\r
-\r
-                  } catch (NumberFormatException e) {\r
-                     e.printStackTrace();\r
-                  } catch (SQLException e) {\r
-                    e.printStackTrace();\r
-                 }\r
+        if (req.getParameter(GROUPID) != null && req.getParameter(FEEDID) == null) {\r
+            StringBuilder groupid1;\r
+            try {\r
+                groupid1 = this.getFeedIdsByGroupId(Integer.parseInt(req.getParameter(GROUPID)));\r
+                map.put(FEEDIDS, groupid1.toString());\r
+            } catch (NumberFormatException | SQLException e) {\r
+                eventlogger.error("PROV0172 StatisticsServlet.doGet: " + e.getMessage(), e);\r
             }\r
-        if(req.getParameter("groupid") != null && req.getParameter("feedid") !=null) {\r
-               StringBuffer groupid1 = new  StringBuffer();\r
-\r
-\r
-                 try {\r
-                     System.out.println("both r not null");\r
-                     groupid1 = this.getFeedIdsByGroupId(Integer.parseInt(req.getParameter("groupid")));\r
-                      System.out.println("feeedids"+req.getParameter("groupid"));\r
-                      groupid1.append(",");\r
-                       groupid1.append(req.getParameter("feedid").replace("|", ",").toString());\r
-\r
-                        map.put("feedids", groupid1.toString());\r
-\r
-\r
-                        System.out.println("groupid1" +groupid1.toString());\r
-\r
-\r
-                  } catch (NumberFormatException e) {\r
-                     e.printStackTrace();\r
-                  } catch (SQLException e) {\r
-                    e.printStackTrace();\r
-                 }\r
+        }\r
+        if (req.getParameter(GROUPID) != null && req.getParameter(FEEDID) != null) {\r
+            StringBuilder groupid1;\r
+            try {\r
+                groupid1 = this.getFeedIdsByGroupId(Integer.parseInt(req.getParameter(GROUPID)));\r
+                groupid1.append(",");\r
+                groupid1.append(req.getParameter(FEEDID).replace("|", ","));\r
+                map.put(FEEDIDS, groupid1.toString());\r
+            } catch (NumberFormatException | SQLException e) {\r
+                eventlogger.error("PROV0173 StatisticsServlet.doGet: " + e.getMessage(), e);\r
             }\r
-\r
-\r
-\r
-        if(req.getParameter("subid")!=null && req.getParameter("feedid") !=null) {\r
-             StringBuffer subidstr = new  StringBuffer();\r
-//             subidstr.append(" and e.DELIVERY_SUBID in(subid)");\r
-//              subidstr.append(req.getParameter("subid").replace("|", ",").toString());\r
-             subidstr.append("and e.DELIVERY_SUBID in(");\r
-\r
-             subidstr.append(req.getParameter("subid").replace("|", ",").toString());\r
-             subidstr.append(")");\r
-             map.put("subid", subidstr.toString());\r
         }\r
-        if(req.getParameter("subid")!=null && req.getParameter("groupid") !=null) {\r
-             StringBuffer subidstr = new  StringBuffer();\r
-//             subidstr.append(" and e.DELIVERY_SUBID in(subid)");\r
-//              subidstr.append(req.getParameter("subid").replace("|", ",").toString());\r
-             subidstr.append("and e.DELIVERY_SUBID in(");\r
-\r
-             subidstr.append(req.getParameter("subid").replace("|", ",").toString());\r
-             subidstr.append(")");\r
-             map.put("subid", subidstr.toString());\r
+        if (req.getParameter(SUBID) != null && req.getParameter(FEEDID) != null) {\r
+            String subidstr = "and e.DELIVERY_SUBID in("\r
+                + req.getParameter(SUBID).replace("|", ",") + ")";\r
+            map.put(SUBID, subidstr);\r
         }\r
-        if(req.getParameter("type")!=null) {\r
-            map.put("eventType", req.getParameter("type").replace("|", ",").toString());\r
+        if (req.getParameter(SUBID) != null && req.getParameter(GROUPID) != null) {\r
+            String subidstr = "and e.DELIVERY_SUBID in("\r
+                + req.getParameter(SUBID).replace("|", ",") + ")";\r
+            map.put(SUBID, subidstr);\r
         }\r
-            if(req.getParameter("output_type")!=null) {\r
-            map.put("output_type", req.getParameter("output_type").toString());\r
+        if (req.getParameter("type") != null) {\r
+            map.put(EVENT_TYPE, req.getParameter("type").replace("|", ","));\r
         }\r
-        if(req.getParameter("start_time")!=null) {\r
-            map.put("start_time", req.getParameter("start_time").toString());\r
+        if (req.getParameter(OUTPUT_TYPE) != null) {\r
+            map.put(OUTPUT_TYPE, req.getParameter(OUTPUT_TYPE));\r
         }\r
-        if(req.getParameter("end_time")!=null) {\r
-            map.put("end_time", req.getParameter("end_time").toString());\r
+        if (req.getParameter(START_TIME) != null) {\r
+            map.put(START_TIME, req.getParameter(START_TIME));\r
         }\r
-\r
-        if(req.getParameter("time")!=null) {\r
-            map.put("start_time", req.getParameter("time").toString());\r
-            map.put("end_time", null);\r
-            }\r
-\r
-\r
-\r
-        if(req.getParameter("output_type") !=null)\r
-        {\r
-            outputType = req.getParameter("output_type");\r
+        if (req.getParameter(END_TIME) != null) {\r
+            map.put(END_TIME, req.getParameter(END_TIME));\r
         }\r
-\r
-\r
-        try {\r
-\r
-            String filterQuery = this.queryGeneretor(map);\r
-            eventlogger.debug("SQL Query for Statistics resultset. "+filterQuery);\r
-\r
-            ResultSet rs=this.getRecordsForSQL(filterQuery);\r
-\r
-            if(outputType.equals("csv")) {\r
-                resp.setContentType("application/octet-stream");\r
-                Date date = new Date() ;\r
-                SimpleDateFormat dateFormat = new SimpleDateFormat("dd-MM-YYYY HH:mm:ss") ;\r
-                resp.setHeader("Content-Disposition", "attachment; filename=\"result:"+dateFormat.format(date)+".csv\"");\r
-                eventlogger.info("Generating CSV file from Statistics resultset");\r
-\r
-                rsToCSV(rs, out);\r
-            }\r
-            else {\r
-                eventlogger.info("Generating JSON for Statistics resultset");\r
-                this.rsToJson(rs, out);\r
-            }\r
+        if (req.getParameter("time") != null) {\r
+            map.put(START_TIME, req.getParameter("time"));\r
+            map.put(END_TIME, null);\r
         }\r
-        catch (IOException e) {\r
-            eventlogger.error("IOException - Generating JSON/CSV:"+e);\r
-            e.printStackTrace();\r
-         }\r
-        catch (JSONException e) {\r
-            eventlogger.error("JSONException - executing SQL query:"+e);\r
-            e.printStackTrace();\r
-        } catch (SQLException e) {\r
-            eventlogger.error("SQLException - executing SQL query:"+e);\r
-            e.printStackTrace();\r
-        } catch (ParseException e) {\r
-            eventlogger.error("ParseException - executing SQL query:"+e);\r
-            e.printStackTrace();\r
+        if (req.getParameter(OUTPUT_TYPE) != null) {\r
+            outputType = req.getParameter(OUTPUT_TYPE);\r
         }\r
+        try {\r
+            this.getRecordsForSQL(map, outputType, resp.getOutputStream(), resp);\r
+        } catch (IOException ioe) {\r
+            eventlogger.error("PROV0174 StatisticsServlet.doGet: " +  ioe.getMessage(), ioe);\r
+        }\r
+\r
     }\r
 \r
 \r
     /**\r
      * rsToJson - Converting RS to JSON object\r
-     * @exception IOException, SQLException\r
+     *\r
      * @param out ServletOutputStream, rs as ResultSet\r
+     * @throws IOException, SQLException\r
      */\r
     public void rsToCSV(ResultSet rs, ServletOutputStream out) throws IOException, SQLException {\r
-        String header = "FEEDNAME,FEEDID,FILES_PUBLISHED,PUBLISH_LENGTH, FILES_DELIVERED, DELIVERED_LENGTH, SUBSCRIBER_URL, SUBID, PUBLISH_TIME,DELIVERY_TIME, AverageDelay\n";\r
-\r
-        // String header = "FEEDNAME,FEEDID,TYPE,REMOTE_ADDR,DELIVERY_SUBID,REQURI,TOTAL CONTENT LENGTH,NO OF FILE,AVERAGE DELAY\n";\r
-\r
-         out.write(header.getBytes());\r
-\r
-         while(rs.next()) {\r
-            StringBuffer line = new StringBuffer();\r
-                line.append(rs.getString("FEEDNAME"));\r
-                line.append(",");\r
-                line.append(rs.getString("FEEDID"));\r
-                line.append(",");\r
-                line.append(rs.getString("FILES_PUBLISHED"));\r
-                line.append(",");\r
-                line.append(rs.getString("PUBLISH_LENGTH"));\r
-                line.append(",");\r
-                line.append(rs.getString("FILES_DELIVERED"));\r
-                line.append(",");\r
-                line.append(rs.getString("DELIVERED_LENGTH"));\r
-                line.append(",");\r
-                line.append(rs.getString("SUBSCRIBER_URL"));\r
-                line.append(",");\r
-                line.append(rs.getString("SUBID"));\r
-                line.append(",");\r
-                line.append(rs.getString("PUBLISH_TIME"));\r
-                line.append(",");\r
-                line.append(rs.getString("DELIVERY_TIME"));\r
-                line.append(",");\r
-                line.append(rs.getString("AverageDelay"));\r
-                line.append(",");\r
-\r
-                line.append("\n");\r
-                out.write(line.toString().getBytes());\r
-                out.flush();\r
-         }\r
+        String header = "FEEDNAME,FEEDID,FILES_PUBLISHED,PUBLISH_LENGTH, FILES_DELIVERED, "\r
+            + "DELIVERED_LENGTH, SUBSCRIBER_URL, SUBID, PUBLISH_TIME,DELIVERY_TIME, AverageDelay\n";\r
+        out.write(header.getBytes());\r
+\r
+        while (rs.next()) {\r
+            String line = rs.getString("FEEDNAME")\r
+                + ","\r
+                + rs.getString(FEEDID)\r
+                + ","\r
+                + rs.getString("FILES_PUBLISHED")\r
+                + ","\r
+                + rs.getString("PUBLISH_LENGTH")\r
+                + ","\r
+                + rs.getString("FILES_DELIVERED")\r
+                + ","\r
+                + rs.getString("DELIVERED_LENGTH")\r
+                + ","\r
+                + rs.getString("SUBSCRIBER_URL")\r
+                + ","\r
+                + rs.getString("SUBID")\r
+                + ","\r
+                + rs.getString("PUBLISH_TIME")\r
+                + ","\r
+                + rs.getString("DELIVERY_TIME")\r
+                + ","\r
+                + rs.getString("AverageDelay")\r
+                + ","\r
+                + "\n";\r
+            out.write(line.getBytes());\r
+            out.flush();\r
+        }\r
     }\r
 \r
     /**\r
      * rsToJson - Converting RS to JSON object\r
-     * @exception IOException, SQLException\r
+     *\r
      * @param out ServletOutputStream, rs as ResultSet\r
+     * @throws IOException, SQLException\r
      */\r
-    public void rsToJson(ResultSet rs, ServletOutputStream out) throws IOException, SQLException {\r
-\r
-        String fields[] = {"FEEDNAME","FEEDID","FILES_PUBLISHED","PUBLISH_LENGTH", "FILES_DELIVERED", "DELIVERED_LENGTH", "SUBSCRIBER_URL", "SUBID", "PUBLISH_TIME","DELIVERY_TIME", "AverageDelay"};\r
-        StringBuffer line = new StringBuffer();\r
-\r
-         line.append("[\n");\r
-\r
-         while(rs.next()) {\r
-             LOGJSONObject j2 = new LOGJSONObject();\r
-             for (String key : fields) {\r
+    private void rsToJson(ResultSet rs, ServletOutputStream out) throws IOException, SQLException {\r
+        String[] fields = {"FEEDNAME", FEEDID, "FILES_PUBLISHED", "PUBLISH_LENGTH", "FILES_DELIVERED",\r
+            "DELIVERED_LENGTH", "SUBSCRIBER_URL", "SUBID", "PUBLISH_TIME", "DELIVERY_TIME",\r
+            "AverageDelay"};\r
+        StringBuilder line = new StringBuilder();\r
+        line.append("[\n");\r
+        while (rs.next()) {\r
+            LOGJSONObject j2 = new LOGJSONObject();\r
+            for (String key : fields) {\r
                 Object v = rs.getString(key);\r
-                if (v != null)\r
+                if (v != null) {\r
                     j2.put(key.toLowerCase(), v);\r
-                else\r
+                } else {\r
                     j2.put(key.toLowerCase(), "");\r
+                }\r
             }\r
-            line =  line.append(j2.toString());;\r
+            line.append(j2.toString());\r
             line.append(",\n");\r
-         }\r
-         line.append("]");\r
+        }\r
+        line.append("]");\r
         out.print(line.toString());\r
     }\r
 \r
     /**\r
      * getFeedIdsByGroupId - Getting FEEDID's by GROUP ID.\r
-     * @exception SQL Query SQLException.\r
-     * @param groupIds\r
+     *\r
+     * @throws SQLException Query SQLException.\r
      */\r
-    public StringBuffer getFeedIdsByGroupId(int groupIds) throws SQLException{\r
-\r
+    private StringBuilder getFeedIdsByGroupId(int groupIds) throws SQLException {\r
         DB db = null;\r
         Connection conn = null;\r
-        PreparedStatement prepareStatement = null;\r
-        ResultSet resultSet=null;\r
+        ResultSet resultSet = null;\r
         String sqlGoupid = null;\r
-        StringBuffer feedIds = new StringBuffer();\r
-\r
+        StringBuilder feedIds = new StringBuilder();\r
         try {\r
             db = new DB();\r
             conn = db.getConnection();\r
-            sqlGoupid= " SELECT FEEDID from FEEDS  WHERE GROUPID = ?";\r
-            prepareStatement =conn.prepareStatement(sqlGoupid);\r
-            prepareStatement.setInt(1, groupIds);\r
-            resultSet=prepareStatement.executeQuery();\r
-            while(resultSet.next()){\r
-                feedIds.append(resultSet.getInt("FEEDID"));\r
-                feedIds.append(",");\r
+            sqlGoupid = " SELECT FEEDID from FEEDS  WHERE GROUPID = ?";\r
+            try (PreparedStatement prepareStatement = conn.prepareStatement(sqlGoupid)) {\r
+                prepareStatement.setInt(1, groupIds);\r
+                resultSet = prepareStatement.executeQuery();\r
+                while (resultSet.next()) {\r
+                    feedIds.append(resultSet.getInt(FEEDID));\r
+                    feedIds.append(",");\r
+                }\r
+                feedIds.deleteCharAt(feedIds.length() - 1);\r
+                eventlogger.info("PROV0177 StatisticsServlet.getFeedIdsByGroupId: feedIds = " + feedIds.toString());\r
             }\r
-            feedIds.deleteCharAt(feedIds.length()-1);\r
-        System.out.println("feedIds"+feedIds.toString());\r
-\r
         } catch (SQLException e) {\r
-            e.printStackTrace();\r
+            eventlogger.error("PROV0175 StatisticsServlet.getFeedIdsByGroupId: " + e.getMessage(), e);\r
         } finally {\r
             try {\r
-                    if(resultSet != null) {\r
-                        resultSet.close();\r
-                        resultSet = null;\r
-                    }\r
-\r
-                    if(prepareStatement != null) {\r
-                        prepareStatement.close();\r
-                        prepareStatement = null;\r
-                    }\r
-\r
-                    if(conn != null){\r
-                        db.release(conn);\r
-                    }\r
-                } catch(Exception e) {\r
-                    e.printStackTrace();\r
+                if (resultSet != null) {\r
+                    resultSet.close();\r
                 }\r
+                if (conn != null) {\r
+                    db.release(conn);\r
+                }\r
+            } catch (Exception e) {\r
+                eventlogger.error("PROV0176 StatisticsServlet.getFeedIdsByGroupId: " + e.getMessage(), e);\r
+            }\r
         }\r
         return feedIds;\r
     }\r
@@ -356,68 +272,85 @@ public class StatisticsServlet extends BaseServlet {
 \r
     /**\r
      * queryGeneretor - Generating sql query\r
-     * @exception ParseException\r
+     *\r
      * @param map as key value pare of all user input fields\r
      */\r
-    public String queryGeneretor(Map<String, String> map) throws ParseException{\r
+    private String queryGeneretor(Map<String, String> map) throws ParseException {\r
 \r
-        String sql = null;\r
+        String sql;\r
         String eventType = null;\r
         String feedids = null;\r
-        String start_time = null;\r
-        String end_time = null;\r
-        String subid=" ";\r
-        if(map.get("eventType") != null){\r
-            eventType=(String) map.get("eventType");\r
+        String startTime = null;\r
+        String endTime = null;\r
+        String subid = " ";\r
+        if (map.get(EVENT_TYPE) != null) {\r
+            eventType =  map.get(EVENT_TYPE);\r
         }\r
-        if(map.get("feedids") != null){\r
-            feedids=(String) map.get("feedids");\r
+        if (map.get(FEEDIDS) != null) {\r
+            feedids = map.get(FEEDIDS);\r
         }\r
-        if(map.get("start_time") != null){\r
-            start_time=(String) map.get("start_time");\r
+        if (map.get(START_TIME) != null) {\r
+            startTime = map.get(START_TIME);\r
         }\r
-        if(map.get("end_time") != null){\r
-            end_time=(String) map.get("end_time");\r
+        if (map.get(END_TIME) != null) {\r
+            endTime =  map.get(END_TIME);\r
         }\r
-        if("all".equalsIgnoreCase(eventType)){\r
-            eventType="PUB','DEL, EXP, PBF";\r
+        if ("all".equalsIgnoreCase(eventType)) {\r
+            eventType = "PUB','DEL, EXP, PBF";\r
         }\r
-        if(map.get("subid") != null){\r
-            subid=(String) map.get("subid");\r
+        if (map.get(SUBID) != null) {\r
+            subid = map.get(SUBID);\r
         }\r
 \r
         eventlogger.info("Generating sql query to get Statistics resultset. ");\r
 \r
-        if(end_time==null && start_time==null ){\r
-\r
+        if (endTime == null && startTime == null) {\r
 \r
-                sql="SELECT (SELECT NAME FROM FEEDS AS f WHERE f.FEEDID in("+feedids+") and f.FEEDID=e.FEEDID) AS FEEDNAME, e.FEEDID as FEEDID, (SELECT COUNT(*) FROM LOG_RECORDS AS c WHERE c.FEEDID in("+feedids+") and c.FEEDID=e.FEEDID AND c.TYPE='PUB') AS FILES_PUBLISHED,(SELECT SUM(content_length) FROM LOG_RECORDS AS c WHERE c.FEEDID in("+feedids+")  and c.FEEDID=e.FEEDID AND c.TYPE='PUB') AS PUBLISH_LENGTH, COUNT(e.EVENT_TIME) as FILES_DELIVERED,  sum(m.content_length) as DELIVERED_LENGTH,SUBSTRING_INDEX(e.REQURI,'/',+3) as SUBSCRIBER_URL, e.DELIVERY_SUBID as SUBID, e.EVENT_TIME AS PUBLISH_TIME, m.EVENT_TIME AS DELIVERY_TIME,  AVG(e.EVENT_TIME - m.EVENT_TIME)/1000 as AverageDelay FROM LOG_RECORDS e JOIN LOG_RECORDS m ON m.PUBLISH_ID = e.PUBLISH_ID AND e.FEEDID IN ("+feedids+") "+subid+" AND m.STATUS=204 AND e.RESULT=204  group by SUBID";\r
+            sql = "SELECT (SELECT NAME FROM FEEDS AS f WHERE f.FEEDID in(" + feedids\r
+                + ") and f.FEEDID=e.FEEDID) AS FEEDNAME, e.FEEDID as FEEDID, (SELECT COUNT(*) FROM LOG_RECORDS AS c WHERE c.FEEDID in("\r
+                + feedids\r
+                + ") and c.FEEDID=e.FEEDID AND c.TYPE='PUB') AS FILES_PUBLISHED,(SELECT SUM(content_length) FROM LOG_RECORDS AS c WHERE c.FEEDID in("\r
+                + feedids\r
+                + ")  and c.FEEDID=e.FEEDID AND c.TYPE='PUB') AS PUBLISH_LENGTH, COUNT(e.EVENT_TIME) as FILES_DELIVERED,  sum(m.content_length) as DELIVERED_LENGTH,SUBSTRING_INDEX(e.REQURI,'/',+3) as SUBSCRIBER_URL, e.DELIVERY_SUBID as SUBID, e.EVENT_TIME AS PUBLISH_TIME, m.EVENT_TIME AS DELIVERY_TIME,  AVG(e.EVENT_TIME - m.EVENT_TIME)/1000 as AverageDelay FROM LOG_RECORDS e JOIN LOG_RECORDS m ON m.PUBLISH_ID = e.PUBLISH_ID AND e.FEEDID IN ("\r
+                + feedids + ") " + subid + " AND m.STATUS=204 AND e.RESULT=204  group by SUBID";\r
 \r
             return sql;\r
-        }else if(start_time!=null && end_time==null ){\r
+        } else if (startTime != null && endTime == null) {\r
 \r
-            long inputTimeInMilli=60000*Long.parseLong(start_time);\r
+            long inputTimeInMilli = 60000 * Long.parseLong(startTime);\r
             Calendar cal = Calendar.getInstance(TimeZone.getTimeZone("GMT"));\r
-            long currentTimeInMilli=cal.getTimeInMillis();\r
-            long compareTime=currentTimeInMilli-inputTimeInMilli;\r
+            long currentTimeInMilli = cal.getTimeInMillis();\r
+            long compareTime = currentTimeInMilli - inputTimeInMilli;\r
+\r
+            sql = "SELECT (SELECT NAME FROM FEEDS AS f WHERE f.FEEDID in(" + feedids\r
+                + ") and f.FEEDID=e.FEEDID) AS FEEDNAME, e.FEEDID as FEEDID, (SELECT COUNT(*) FROM LOG_RECORDS AS c WHERE c.FEEDID in("\r
+                + feedids\r
+                + ") and c.FEEDID=e.FEEDID AND c.TYPE='PUB') AS FILES_PUBLISHED,(SELECT SUM(content_length) FROM LOG_RECORDS AS c WHERE c.FEEDID in("\r
+                + feedids\r
+                + ")  and c.FEEDID=e.FEEDID AND c.TYPE='PUB') AS PUBLISH_LENGTH, COUNT(e.EVENT_TIME) as FILES_DELIVERED,  sum(m.content_length) as DELIVERED_LENGTH,SUBSTRING_INDEX(e.REQURI,'/',+3) as SUBSCRIBER_URL, e.DELIVERY_SUBID as SUBID, e.EVENT_TIME AS PUBLISH_TIME, m.EVENT_TIME AS DELIVERY_TIME,  AVG(e.EVENT_TIME - m.EVENT_TIME)/1000 as AverageDelay FROM LOG_RECORDS e JOIN LOG_RECORDS m ON m.PUBLISH_ID = e.PUBLISH_ID AND e.FEEDID IN ("\r
+                + feedids + ") " + subid + " AND m.STATUS=204 AND e.RESULT=204 and e.event_time>="\r
+                + compareTime + " group by SUBID";\r
 \r
-              sql="SELECT (SELECT NAME FROM FEEDS AS f WHERE f.FEEDID in("+feedids+") and f.FEEDID=e.FEEDID) AS FEEDNAME, e.FEEDID as FEEDID, (SELECT COUNT(*) FROM LOG_RECORDS AS c WHERE c.FEEDID in("+feedids+") and c.FEEDID=e.FEEDID AND c.TYPE='PUB') AS FILES_PUBLISHED,(SELECT SUM(content_length) FROM LOG_RECORDS AS c WHERE c.FEEDID in("+feedids+")  and c.FEEDID=e.FEEDID AND c.TYPE='PUB') AS PUBLISH_LENGTH, COUNT(e.EVENT_TIME) as FILES_DELIVERED,  sum(m.content_length) as DELIVERED_LENGTH,SUBSTRING_INDEX(e.REQURI,'/',+3) as SUBSCRIBER_URL, e.DELIVERY_SUBID as SUBID, e.EVENT_TIME AS PUBLISH_TIME, m.EVENT_TIME AS DELIVERY_TIME,  AVG(e.EVENT_TIME - m.EVENT_TIME)/1000 as AverageDelay FROM LOG_RECORDS e JOIN LOG_RECORDS m ON m.PUBLISH_ID = e.PUBLISH_ID AND e.FEEDID IN ("+feedids+") "+subid+" AND m.STATUS=204 AND e.RESULT=204 and e.event_time>="+compareTime+" group by SUBID";\r
-\r
-             return sql;\r
+            return sql;\r
 \r
-        }else{\r
+        } else {\r
             SimpleDateFormat inFormat = new SimpleDateFormat("yyyy-MM-dd'T'HH:mm:ss");\r
-            Date startDate=inFormat.parse(start_time);\r
-            Date endDate=inFormat.parse(end_time);\r
+            Date startDate = inFormat.parse(startTime);\r
+            Date endDate = inFormat.parse(endTime);\r
 \r
-            long startInMillis=startDate.getTime();\r
-            long endInMillis=endDate.getTime();\r
+            long startInMillis = startDate.getTime();\r
+            long endInMillis = endDate.getTime();\r
 \r
-             {\r
+            sql = "SELECT (SELECT NAME FROM FEEDS AS f WHERE f.FEEDID in(" + feedids\r
+                + ") and f.FEEDID=e.FEEDID) AS FEEDNAME, e.FEEDID as FEEDID, (SELECT COUNT(*) FROM LOG_RECORDS AS c WHERE c.FEEDID in("\r
+                + feedids\r
+                + ") and c.FEEDID=e.FEEDID AND c.TYPE='PUB') AS FILES_PUBLISHED,(SELECT SUM(content_length) FROM LOG_RECORDS AS c WHERE c.FEEDID in("\r
+                + feedids\r
+                + ")  and c.FEEDID=e.FEEDID AND c.TYPE='PUB') AS PUBLISH_LENGTH, COUNT(e.EVENT_TIME) as FILES_DELIVERED,  sum(m.content_length) as DELIVERED_LENGTH,SUBSTRING_INDEX(e.REQURI,'/',+3) as SUBSCRIBER_URL, e.DELIVERY_SUBID as SUBID, e.EVENT_TIME AS PUBLISH_TIME, m.EVENT_TIME AS DELIVERY_TIME,  AVG(e.EVENT_TIME - m.EVENT_TIME)/1000 as AverageDelay FROM LOG_RECORDS e JOIN LOG_RECORDS m ON m.PUBLISH_ID = e.PUBLISH_ID AND e.FEEDID IN ("\r
+                + feedids + ") " + subid + " AND m.STATUS=204 AND e.RESULT=204 and e.event_time between " + startInMillis\r
+                + " and " + endInMillis + " group by SUBID";\r
 \r
-                sql="SELECT (SELECT NAME FROM FEEDS AS f WHERE f.FEEDID in("+feedids+") and f.FEEDID=e.FEEDID) AS FEEDNAME, e.FEEDID as FEEDID, (SELECT COUNT(*) FROM LOG_RECORDS AS c WHERE c.FEEDID in("+feedids+") and c.FEEDID=e.FEEDID AND c.TYPE='PUB') AS FILES_PUBLISHED,(SELECT SUM(content_length) FROM LOG_RECORDS AS c WHERE c.FEEDID in("+feedids+")  and c.FEEDID=e.FEEDID AND c.TYPE='PUB') AS PUBLISH_LENGTH, COUNT(e.EVENT_TIME) as FILES_DELIVERED,  sum(m.content_length) as DELIVERED_LENGTH,SUBSTRING_INDEX(e.REQURI,'/',+3) as SUBSCRIBER_URL, e.DELIVERY_SUBID as SUBID, e.EVENT_TIME AS PUBLISH_TIME, m.EVENT_TIME AS DELIVERY_TIME,  AVG(e.EVENT_TIME - m.EVENT_TIME)/1000 as AverageDelay FROM LOG_RECORDS e JOIN LOG_RECORDS m ON m.PUBLISH_ID = e.PUBLISH_ID AND e.FEEDID IN ("+feedids+") "+subid+" AND m.STATUS=204 AND e.RESULT=204 and e.event_time between "+startInMillis+" and "+endInMillis+" group by SUBID";\r
 \r
-            }\r
             return sql;\r
         }\r
     }\r
@@ -427,43 +360,45 @@ public class StatisticsServlet extends BaseServlet {
      * PUT a Statistics URL -- not supported.\r
      */\r
     @Override\r
-    public void doPut(HttpServletRequest req, HttpServletResponse resp) throws IOException {\r
+    public void doPut(HttpServletRequest req, HttpServletResponse resp) {\r
         String message = "PUT not allowed for the StatisticsURL.";\r
         EventLogRecord elr = new EventLogRecord(req);\r
         elr.setMessage(message);\r
         elr.setResult(HttpServletResponse.SC_METHOD_NOT_ALLOWED);\r
-        eventlogger.info(elr);\r
-        resp.sendError(HttpServletResponse.SC_METHOD_NOT_ALLOWED, message);\r
+        eventlogger.error(elr.toString());\r
+        sendResponseError(resp, HttpServletResponse.SC_METHOD_NOT_ALLOWED, message, eventlogger);\r
     }\r
+\r
     /**\r
      * POST a Statistics URL -- not supported.\r
      */\r
     @Override\r
-    public void doPost(HttpServletRequest req, HttpServletResponse resp) throws IOException {\r
+    public void doPost(HttpServletRequest req, HttpServletResponse resp) {\r
         String message = "POST not allowed for the StatisticsURL.";\r
         EventLogRecord elr = new EventLogRecord(req);\r
         elr.setMessage(message);\r
         elr.setResult(HttpServletResponse.SC_METHOD_NOT_ALLOWED);\r
-        eventlogger.info(elr);\r
-        resp.sendError(HttpServletResponse.SC_METHOD_NOT_ALLOWED, message);\r
+        eventlogger.error(elr.toString());\r
+        sendResponseError(resp, HttpServletResponse.SC_METHOD_NOT_ALLOWED, message, eventlogger);\r
     }\r
 \r
     private Map<String, String> buildMapFromRequest(HttpServletRequest req) {\r
-        Map<String, String> map = new HashMap<String, String>();\r
+        Map<String, String> map = new HashMap<>();\r
         String s = req.getParameter("type");\r
         if (s != null) {\r
-            if (s.equals("pub") || s.equals("del") || s.equals("exp")) {\r
+            if ("pub".equals(s) || "del".equals(s) || "exp".equals(s)) {\r
                 map.put("type", s);\r
             } else {\r
                 map.put("err", "bad type");\r
                 return map;\r
             }\r
-        } else\r
-        map.put("type", "all");\r
+        } else {\r
+            map.put("type", "all");\r
+        }\r
         map.put("publishSQL", "");\r
         map.put("statusSQL", "");\r
         map.put("resultSQL", "");\r
-        map.put("reasonSQL", "");\r
+        map.put(REASON_SQL, "");\r
 \r
         s = req.getParameter("publishId");\r
         if (s != null) {\r
@@ -471,25 +406,32 @@ public class StatisticsServlet extends BaseServlet {
                 map.put("err", "bad publishId");\r
                 return map;\r
             }\r
-            map.put("publishSQL", " AND PUBLISH_ID = '"+s+"'");\r
+            map.put("publishSQL", " AND PUBLISH_ID = '" + s + "'");\r
         }\r
 \r
         s = req.getParameter("statusCode");\r
         if (s != null) {\r
             String sql = null;\r
-            if (s.equals("success")) {\r
-                sql = " AND STATUS >= 200 AND STATUS < 300";\r
-            } else if (s.equals("redirect")) {\r
-                sql = " AND STATUS >= 300 AND STATUS < 400";\r
-            } else if (s.equals("failure")) {\r
-                sql = " AND STATUS >= 400";\r
-            } else {\r
-                try {\r
-                    Integer n = Integer.parseInt(s);\r
-                    if ((n >= 100 && n < 600) || (n == -1))\r
-                        sql = " AND STATUS = " + n;\r
-                } catch (NumberFormatException e) {\r
-                }\r
+            switch (s) {\r
+                case "success":\r
+                    sql = " AND STATUS >= 200 AND STATUS < 300";\r
+                    break;\r
+                case "redirect":\r
+                    sql = " AND STATUS >= 300 AND STATUS < 400";\r
+                    break;\r
+                case "failure":\r
+                    sql = " AND STATUS >= 400";\r
+                    break;\r
+                default:\r
+                    try {\r
+                        int n = Integer.parseInt(s);\r
+                        if ((n >= 100 && n < 600) || (n == -1)) {\r
+                            sql = " AND STATUS = " + n;\r
+                        }\r
+                    } catch (NumberFormatException e) {\r
+                        eventlogger.error("Failed to parse input", e);\r
+                    }\r
+                    break;\r
             }\r
             if (sql == null) {\r
                 map.put("err", "bad statusCode");\r
@@ -502,17 +444,22 @@ public class StatisticsServlet extends BaseServlet {
         s = req.getParameter("expiryReason");\r
         if (s != null) {\r
             map.put("type", "exp");\r
-            if (s.equals("notRetryable")) {\r
-                map.put("reasonSQL", " AND REASON = 'notRetryable'");\r
-            } else if (s.equals("retriesExhausted")) {\r
-                map.put("reasonSQL", " AND REASON = 'retriesExhausted'");\r
-            } else if (s.equals("diskFull")) {\r
-                map.put("reasonSQL", " AND REASON = 'diskFull'");\r
-            } else if (s.equals("other")) {\r
-                map.put("reasonSQL", " AND REASON = 'other'");\r
-            } else {\r
-                map.put("err", "bad expiryReason");\r
-                return map;\r
+            switch (s) {\r
+                case "notRetryable":\r
+                    map.put(REASON_SQL, " AND REASON = 'notRetryable'");\r
+                    break;\r
+                case "retriesExhausted":\r
+                    map.put(REASON_SQL, " AND REASON = 'retriesExhausted'");\r
+                    break;\r
+                case "diskFull":\r
+                    map.put(REASON_SQL, " AND REASON = 'diskFull'");\r
+                    break;\r
+                case "other":\r
+                    map.put(REASON_SQL, " AND REASON = 'other'");\r
+                    break;\r
+                default:\r
+                    map.put("err", "bad expiryReason");\r
+                    return map;\r
             }\r
         }\r
 \r
@@ -537,52 +484,64 @@ public class StatisticsServlet extends BaseServlet {
         map.put("timeSQL", String.format(" AND EVENT_TIME >= %d AND EVENT_TIME <= %d", stime, etime));\r
         return map;\r
     }\r
+\r
     private long getTimeFromParam(final String s) {\r
-        if (s == null)\r
+        if (s == null) {\r
             return 0;\r
+        }\r
         try {\r
             // First, look for an RFC 3339 date\r
-            String fmt = (s.indexOf('.') > 0) ? fmt2 : fmt1;\r
+            String fmt = (s.indexOf('.') > 0) ? FMT2 : FMT1;\r
             SimpleDateFormat sdf = new SimpleDateFormat(fmt);\r
             Date d = sdf.parse(s);\r
             return d.getTime();\r
         } catch (ParseException e) {\r
+            intlogger.error("Exception in getting Time :- " + e.getMessage(), e);\r
         }\r
         try {\r
             // Also allow a long (in ms); useful for testing\r
-            long n = Long.parseLong(s);\r
-            return n;\r
+            return Long.parseLong(s);\r
         } catch (NumberFormatException e) {\r
+            intlogger.error("Exception in getting Time :- " + e.getMessage(), e);\r
         }\r
-        intlogger.info("Error parsing time="+s);\r
+        intlogger.info("Error parsing time=" + s);\r
         return -1;\r
     }\r
 \r
-\r
-    private ResultSet getRecordsForSQL(String sql) {\r
-        intlogger.debug(sql);\r
-        long start = System.currentTimeMillis();\r
-        DB db = new DB();\r
-        Connection conn = null;\r
-        ResultSet rs=null;\r
-\r
+    private void getRecordsForSQL(Map<String, String> map, String outputType, ServletOutputStream out, HttpServletResponse resp) {\r
         try {\r
-            conn = db.getConnection();\r
-            Statement  stmt = conn.createStatement();\r
-            PreparedStatement pst=conn.prepareStatement(sql);\r
-            rs=pst.executeQuery();\r
-            //this.rsToJson(rs)\r
-            //rs.close();\r
-            stmt.close();\r
-        } catch (SQLException e) {\r
-            e.printStackTrace();\r
-        } finally {\r
-            if (conn != null)\r
+            String filterQuery = this.queryGeneretor(map);\r
+            eventlogger.debug("SQL Query for Statistics resultset. " + filterQuery);\r
+            intlogger.debug(filterQuery);\r
+            long start = System.currentTimeMillis();\r
+            DB db = new DB();\r
+            try (Connection conn = db.getConnection()) {\r
+                try (ResultSet rs = conn.prepareStatement(filterQuery).executeQuery()) {\r
+                    if ("csv".equals(outputType)) {\r
+                        resp.setContentType("application/octet-stream");\r
+                        Date date = new Date();\r
+                        SimpleDateFormat dateFormat = new SimpleDateFormat("dd-MM-YYYY HH:mm:ss");\r
+                        resp.setHeader("Content-Disposition",\r
+                            "attachment; filename=\"result:" + dateFormat.format(date) + ".csv\"");\r
+                        eventlogger.info("Generating CSV file from Statistics resultset");\r
+                        rsToCSV(rs, out);\r
+                    } else {\r
+                        eventlogger.info("Generating JSON for Statistics resultset");\r
+                        this.rsToJson(rs, out);\r
+                    }\r
+                }\r
                 db.release(conn);\r
+            } catch (SQLException e) {\r
+                eventlogger.error("SQLException:" + e);\r
+            }\r
+            intlogger.debug("Time: " + (System.currentTimeMillis() - start) + " ms");\r
+        } catch (IOException e) {\r
+            eventlogger.error("IOException - Generating JSON/CSV:" + e);\r
+        } catch (JSONException e) {\r
+            eventlogger.error("JSONException - executing SQL query:" + e);\r
+        } catch (ParseException e) {\r
+            eventlogger.error("ParseException - executing SQL query:" + e);\r
         }\r
-\r
-        intlogger.debug("Time: " + (System.currentTimeMillis()-start) + " ms");\r
-\r
-        return rs;\r
     }\r
 }\r
+\r