bind session and make defensive copy of sublist

reiern70 · Nov 26, 2014 · 91bc263 · 91bc263
1 parent 74924af
commit 91bc263
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/bgprocess/src/main/java/com/antilia/BgProcessSession.java b/bgprocess/src/main/java/com/antilia/BgProcessSession.java
@@ -23,7 +23,8 @@ public BgProcessSession(Request request) {
 		super(request);
 	}
 
-	public void add(ExecutionBridge bridge) {
+	public synchronized void add(ExecutionBridge bridge) {
+		bind();
 		bridges.add(bridge);
 	}
 
@@ -39,8 +40,7 @@ public synchronized void  pruneFinishedTasks() {
 
 	public Iterator<ExecutionBridge> getTasksPage(int start, int size) {
 		int min = Math.min(size, bridges.size());
-		// mgrigorov: maybe make a copy of the subList to avoid ConcurrentModificationExceptions ?!
-		return bridges.subList(start, min).iterator();
+		return new ArrayList<ExecutionBridge>(bridges.subList(start, min)).iterator();
 	}
 
 	public long countTasks() {