using statistics with synchronized methods (#1563)

Author: tulinkry

src/org/opensolaris/opengrok/web/Statistics.java

@@ -67,7 +67,7 @@ public class Statistics {
     /**
      * Adds a single request into all requests.
      */
-    public void addRequest(HttpServletRequest req) {
+    synchronized public void addRequest(HttpServletRequest req) {
         maybeRefresh();
 
         requestsPerMinute++;
@@ -87,7 +87,7 @@ public void addRequest(HttpServletRequest req) {
     /**
      * Refreshes the last timestamp and number of minutes since start if needed.
      */
-    protected void maybeRefresh() {
+    synchronized protected void maybeRefresh() {
         if (timeStart + 60 * 1000 <= System.currentTimeMillis()) {
             // several minutes have passed
             minutes += (System.currentTimeMillis() - timeStart) / (60 * 1000);
@@ -102,7 +102,7 @@ protected void maybeRefresh() {
      * @param req the given request
      * @param category category
      */
-    public void addRequest(HttpServletRequest req, String category) {
+    synchronized public void addRequest(HttpServletRequest req, String category) {
         Long val = requestCategories.get(category);
         if (val == null) {
             val = new Long(0);
@@ -118,7 +118,7 @@ public void addRequest(HttpServletRequest req, String category) {
      * @param category category
      * @param v time spent on processing this request
      */
-    public void addRequestTime(HttpServletRequest req, String category, long v) {
+    synchronized public void addRequestTime(HttpServletRequest req, String category, long v) {
         addRequest(req, category);
         Long val = timing.get(category);
         Long min = timingMin.get(category);
@@ -172,35 +172,35 @@ public Map<String, Double> getTimingAvg() {
         return timingAvg;
     }
 
-    public void setRequestCategories(Map<String, Long> requestCategories) {
+    synchronized public void setRequestCategories(Map<String, Long> requestCategories) {
         this.requestCategories = requestCategories;
     }
 
-    public void setTiming(Map<String, Long> timing) {
+    synchronized public void setTiming(Map<String, Long> timing) {
         this.timing = timing;
     }
 
-    public void setTimingMin(Map<String, Long> timing_min) {
+    synchronized public void setTimingMin(Map<String, Long> timing_min) {
         this.timingMin = timing_min;
     }
 
-    public void setTimingMax(Map<String, Long> timing_max) {
+    synchronized public void setTimingMax(Map<String, Long> timing_max) {
         this.timingMax = timing_max;
     }
 
     public long getTimeStart() {
         return timeStart;
     }
 
-    public void setTimeStart(long timeStart) {
+    synchronized public void setTimeStart(long timeStart) {
         this.timeStart = timeStart;
     }
 
     public long getRequests() {
         return requests;
     }
 
-    public void setRequests(long requests) {
+    synchronized public void setRequests(long requests) {
         this.requests = requests;
     }
 
@@ -209,7 +209,7 @@ public long getMinutes() {
         return minutes;
     }
 
-    public void setMinutes(long minutes) {
+    synchronized public void setMinutes(long minutes) {
         this.minutes = minutes;
     }
 
@@ -218,7 +218,7 @@ public long getRequestsPerMinute() {
         return requestsPerMinute;
     }
 
-    public void setRequestsPerMinute(long requestsPerMinute) {
+    synchronized public void setRequestsPerMinute(long requestsPerMinute) {
         this.requestsPerMinute = requestsPerMinute;
     }
 
@@ -229,7 +229,7 @@ public long getRequestsPerMinuteMin() {
         return requestsPerMinuteMin;
     }
 
-    public void setRequestsPerMinuteMin(long requestsPerMinuteMin) {
+    synchronized public void setRequestsPerMinuteMin(long requestsPerMinuteMin) {
         this.requestsPerMinuteMin = requestsPerMinuteMin;
     }
 
@@ -240,7 +240,7 @@ public long getRequestsPerMinuteMax() {
         return requestsPerMinuteMax;
     }
 
-    public void setRequestsPerMinuteMax(long requestsPerMinuteMax) {
+    synchronized public void setRequestsPerMinuteMax(long requestsPerMinuteMax) {
         this.requestsPerMinuteMax = requestsPerMinuteMax;
     }
 
@@ -253,15 +253,15 @@ public long[] getDayHistogram() {
         return dayHistogram;
     }
 
-    public void setDayHistogram(long[] dayHistogram) {
+    synchronized public void setDayHistogram(long[] dayHistogram) {
         this.dayHistogram = dayHistogram;
     }
 
     public long[] getMonthHistogram() {
         return monthHistogram;
     }
 
-    public void setMonthHistogram(long[] monthHistogram) {
+    synchronized public void setMonthHistogram(long[] monthHistogram) {
         this.monthHistogram = monthHistogram;
     }