s434730/RandomSec
3
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Synchronize access to processes list - fixes #862

Browse Source
This commit is contained in:
Tom Morris 2014-04-05 12:36:40 -04:00
parent e6bc3a5aa7
commit 672e9bf62c
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
main/src/com/google/refine/process/ProcessManager.java
View File

@ -33,6 +33,7 @@ OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
package com.google.refine.process;
import java.util.Collections;
import java.util.LinkedList;
import java.util.List;
import java.util.Properties;
@ -45,7 +46,7 @@ import com.google.refine.history.HistoryEntry;
import com.google.refine.history.HistoryProcess;
public class ProcessManager implements Jsonizable {
protected List<Process> _processes = new LinkedList<Process>();
protected List<Process> _processes = Collections.synchronizedList(new LinkedList<Process>());
protected List<Exception> _latestExceptions = null;
public ProcessManager() {
@ -58,8 +59,10 @@ public class ProcessManager implements Jsonizable {
writer.object();
writer.key("processes"); writer.array();
for (Process p : _processes) {
p.write(writer, options);
synchronized (_processes) {
for (Process p : _processes) {
p.write(writer, options);
}
}
writer.endArray();