[infinispan-commits] Infinispan SVN: r453 - trunk/core/src/main/java/org/infinispan/container.

[infinispan-commits at lists.jboss.org]

Author: manik.surtani at jboss.com
Date: 2009-06-12 07:38:31 -0400 (Fri, 12 Jun 2009)
New Revision: 453

Modified:
   trunk/core/src/main/java/org/infinispan/container/SimpleDataContainer.java
Log:
reverted

Modified: trunk/core/src/main/java/org/infinispan/container/SimpleDataContainer.java
===================================================================
--- trunk/core/src/main/java/org/infinispan/container/SimpleDataContainer.java	2009-06-12 11:31:36 UTC (rev 452)
+++ trunk/core/src/main/java/org/infinispan/container/SimpleDataContainer.java	2009-06-12 11:38:31 UTC (rev 453)
@@ -189,11 +189,7 @@
       Iterator<Object> currentIterator;
 
       private KeyIterator(Iterator<Object> immortalIterator, Iterator<Object> mortalIterator) {
-         // this order is imperative.  We need to iterate through mortal entries before the immortal ones
-         // since some access patterns (Dist rehashing) iterate through the key set and move entries from
-         // being immortal to mortal.  If you iterate through immortal entries first you could
-         // see the same key twice, breaking Set semantics.
-         metaIterator = Arrays.asList(mortalIterator, immortalIterator).iterator();
+         metaIterator = Arrays.asList(immortalIterator, mortalIterator).iterator();
          if (metaIterator.hasNext()) currentIterator = metaIterator.next();
       }