AbstractAccumulator: synchronized add and null check

Change-Id: I2999311d19ab36c17413ebc93398483a7d012714

@@ -72,9 +72,9 @@ public abstract class AbstractAccumulator<T> implements Accumulator<T> {
     }
     }
 
 
     @Override
     @Override
-    public void add(T event) {
+    public synchronized void add(T event) {
         idleTask = cancelIfActive(idleTask);
         idleTask = cancelIfActive(idleTask);
-        events.add(event);
+        events.add(checkNotNull(event, "Event cannot be null"));
 
 
         // Did we hit the max event threshold?
         // Did we hit the max event threshold?
         if (events.size() == maxEvents) {
         if (events.size() == maxEvents) {
@@ -114,7 +114,7 @@ public abstract class AbstractAccumulator<T> implements Accumulator<T> {
                 maxTask = cancelIfActive(maxTask);
                 maxTask = cancelIfActive(maxTask);
                 processEvents(finalizeCurrentBatch());
                 processEvents(finalizeCurrentBatch());
             } catch (Exception e) {
             } catch (Exception e) {
-                log.warn("Unable to process batch due to {}", e.getMessage());
+                log.warn("Unable to process batch due to {}", e);
             }
             }
         }
         }
     }
     }