[teiid-commits] teiid SVN: r2867 - branches/7.1.x/client/src/main/java/org/teiid/adminapi/impl.

[teiid-commits at lists.jboss.org]

Author: rareddy
Date: 2011-01-20 22:35:46 -0500 (Thu, 20 Jan 2011)
New Revision: 2867

Modified:
   branches/7.1.x/client/src/main/java/org/teiid/adminapi/impl/ModelMetaData.java
Log:
TEIID-1445: fixing the ConcurrentModificationException  with error list by synchronizing it.

Modified: branches/7.1.x/client/src/main/java/org/teiid/adminapi/impl/ModelMetaData.java
===================================================================
--- branches/7.1.x/client/src/main/java/org/teiid/adminapi/impl/ModelMetaData.java	2011-01-21 01:40:16 UTC (rev 2866)
+++ branches/7.1.x/client/src/main/java/org/teiid/adminapi/impl/ModelMetaData.java	2011-01-21 03:35:46 UTC (rev 2867)
@@ -214,7 +214,7 @@
 		this.errors = errors;
 	}	
 	
-	public List<ValidationError> getValidationErrors(ValidationError.Severity severity){
+	public synchronized List<ValidationError> getValidationErrors(ValidationError.Severity severity){
 		if (this.errors == null) {
 			return Collections.emptyList();
 		}
@@ -227,7 +227,7 @@
 		return list;
 	}	
 	
-    public ValidationError addError(String severity, String message) {
+    public synchronized ValidationError addError(String severity, String message) {
         if (this.errors == null) {
             this.errors = new ArrayList<ValidationError>();
         }
@@ -236,14 +236,14 @@
         return ve;
     }
     
-    public boolean removeError(ValidationError remove) {
+    public synchronized boolean removeError(ValidationError remove) {
     	if (this.errors == null) {
     		return false;
     	}
     	return this.errors.remove(remove);
     }
     
-    public void clearErrors() {
+    public synchronized void clearErrors() {
     	this.errors.clear();
     }