You are viewing a plain text version of this content. The canonical link for it is here.
Posted to axis-cvs@ws.apache.org by di...@apache.org on 2008/01/16 22:48:30 UTC
svn commit: r612577 -
/webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/engine/DeployableChain.java
Author: dims
Date: Wed Jan 16 13:48:29 2008
New Revision: 612577
URL: http://svn.apache.org/viewvc?rev=612577&view=rev
Log:
HashSet's iterator order is not predictable. Getting a build break on IBM JDK6. So use LinkedHashSet
Modified:
webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/engine/DeployableChain.java
Modified: webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/engine/DeployableChain.java
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/engine/DeployableChain.java?rev=612577&r1=612576&r2=612577&view=diff
==============================================================================
--- webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/engine/DeployableChain.java (original)
+++ webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/engine/DeployableChain.java Wed Jan 16 13:48:29 2008
@@ -27,6 +27,8 @@
import java.util.Set;
import java.util.HashSet;
import java.util.Iterator;
+import java.util.LinkedHashMap;
+import java.util.LinkedHashSet;
/**
* A DeployableChain is a container which manages dependencies between Deployables. You
@@ -109,7 +111,7 @@
if (mySuccessors != null && !mySuccessors.isEmpty()) {
Set successors = (Set)activeConstraints.get(name);
if (successors == null) {
- successors = new HashSet();
+ successors = new LinkedHashSet();
activeConstraints.put(name, successors);
}
successors.addAll(mySuccessors);
@@ -200,7 +202,7 @@
public void rebuild() throws Exception {
chain.clear();
- Set keys = new HashSet();
+ Set keys = new LinkedHashSet();
keys.addAll(deployed.keySet());
// First goes first.
@@ -222,7 +224,7 @@
while (!keys.isEmpty()) {
String name = (String)keys.iterator().next();
- getMinIndex(name, keys, new HashSet());
+ getMinIndex(name, keys, new LinkedHashSet());
}
// Now we've got a chain of names. Convert to actual things before we return.
@@ -241,7 +243,7 @@
public void addRelationship(String before, String after) {
Set successors = (Set)activeConstraints.get(before);
if (successors == null) {
- successors = new HashSet();
+ successors = new LinkedHashSet();
activeConstraints.put(before, successors);
}
successors.add(after);
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-cvs-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-cvs-help@ws.apache.org