# IGNITE-56 Revert GridCachePartitionedReloadAllAbstractSelfTest test.

Project: http://git-wip-us.apache.org/repos/asf/incubator-ignite/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-ignite/commit/068931b3
Tree: http://git-wip-us.apache.org/repos/asf/incubator-ignite/tree/068931b3
Diff: http://git-wip-us.apache.org/repos/asf/incubator-ignite/diff/068931b3

Branch: refs/heads/ignite-181
Commit: 068931b378a49c9a3b2f93d92321182ffe758a38
Parents: db5e5d3
Author: sevdokimov <sergey.evdoki...@jetbrains.com>
Authored: Mon Feb 9 00:28:14 2015 +0300
Committer: sevdokimov <sergey.evdoki...@jetbrains.com>
Committed: Mon Feb 9 00:28:14 2015 +0300

----------------------------------------------------------------------
 ...chePartitionedReloadAllAbstractSelfTest.java | 56 +++++++++++++-------
 1 file changed, 38 insertions(+), 18 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-ignite/blob/068931b3/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/distributed/GridCachePartitionedReloadAllAbstractSelfTest.java
----------------------------------------------------------------------
diff --git 
a/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/distributed/GridCachePartitionedReloadAllAbstractSelfTest.java
 
b/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/distributed/GridCachePartitionedReloadAllAbstractSelfTest.java
index 1797eee..eb1900e 100644
--- 
a/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/distributed/GridCachePartitionedReloadAllAbstractSelfTest.java
+++ 
b/modules/core/src/test/java/org/apache/ignite/internal/processors/cache/distributed/GridCachePartitionedReloadAllAbstractSelfTest.java
@@ -19,7 +19,6 @@ package 
org.apache.ignite.internal.processors.cache.distributed;
 
 import org.apache.ignite.*;
 import org.apache.ignite.cache.*;
-import org.apache.ignite.cache.affinity.*;
 import org.apache.ignite.cache.store.*;
 import org.apache.ignite.configuration.*;
 import org.apache.ignite.internal.util.typedef.*;
@@ -56,7 +55,7 @@ public abstract class 
GridCachePartitionedReloadAllAbstractSelfTest extends Grid
     private final Map<Integer, String> map = new ConcurrentHashMap8<>();
 
     /** Collection of caches, one per grid node. */
-    private List<IgniteCache<Integer, String>> caches;
+    private List<GridCache<Integer, String>> caches;
 
     /** {@inheritDoc} */
     @SuppressWarnings("unchecked")
@@ -130,7 +129,7 @@ public abstract class 
GridCachePartitionedReloadAllAbstractSelfTest extends Grid
         caches = new ArrayList<>(GRID_CNT);
 
         for (int i = 0; i < GRID_CNT; i++)
-            caches.add(startGrid(i).<Integer, String>jcache(null));
+            caches.add(startGrid(i).<Integer, String>cache(null));
 
         awaitPartitionMapExchange();
     }
@@ -156,14 +155,14 @@ public abstract class 
GridCachePartitionedReloadAllAbstractSelfTest extends Grid
 
             @Override public void loadCache(IgniteBiInClosure<Integer, String> 
c,
                 Object... args) {
-                X.println("Loading all on: " + caches.indexOf(g.<Integer, 
String>jcache(null)));
+                X.println("Loading all on: " + caches.indexOf(g.<Integer, 
String>cache(null)));
 
                 for (Map.Entry<Integer, String> e : map.entrySet())
                     c.apply(e.getKey(), e.getValue());
             }
 
             @Override public String load(Integer key) {
-                X.println("Loading on: " + caches.indexOf(g.<Integer, 
String>jcache(null)) + " key=" + key);
+                X.println("Loading on: " + caches.indexOf(g.<Integer, 
String>cache(null)) + " key=" + key);
 
                 return map.get(key);
             }
@@ -186,8 +185,8 @@ public abstract class 
GridCachePartitionedReloadAllAbstractSelfTest extends Grid
      */
     public void testReloadAll() throws Exception {
         // Fill caches with values.
-        for (IgniteCache<Integer, String> cache : caches) {
-            Iterable<Integer> keys = primaryKeys(cache, 100);
+        for (GridCache<Integer, String> cache : caches) {
+            Iterable<Integer> keys = primaryKeysForCache(cache, 100);
 
             info("Values [cache=" + caches.indexOf(cache) + ", size=" + 
F.size(keys.iterator()) +  ", keys=" + keys + "]");
 
@@ -195,33 +194,54 @@ public abstract class 
GridCachePartitionedReloadAllAbstractSelfTest extends Grid
                 map.put(key, "val" + key);
         }
 
-        Collection<IgniteCache<Integer, String>> emptyCaches = new 
ArrayList<>(caches);
+        Collection<GridCache<Integer, String>> emptyCaches = new 
ArrayList<>(caches);
 
-        for (IgniteCache<Integer, String> cache : caches) {
+        for (GridCache<Integer, String> cache : caches) {
             info("Reloading cache: " + caches.indexOf(cache));
 
             // Check data is reloaded only on the nodes on which reloadAll() 
has been called.
             if (!nearEnabled()) {
-                for (IgniteCache<Integer, String> eCache : emptyCaches)
+                for (GridCache<Integer, String> eCache : emptyCaches)
                     assertEquals("Non-null values found in cache [cache=" + 
caches.indexOf(eCache) +
-                        ", size=" + eCache.size() + ", size=" + eCache.size() 
+ "]",
+                        ", size=" + eCache.size() + ", size=" + eCache.size() +
+                        ", entrySetSize=" + eCache.entrySet().size() + "]",
                         0, eCache.size());
             }
 
-            loadAll(cache, map.keySet(), true);
+            cache.reloadAll(map.keySet());
 
             for (Integer key : map.keySet()) {
-                CacheAffinity<Integer> affinity = affinity(cache);
+                CacheEntry entry = cache.entry(key);
 
-                if (affinity.isPrimary(localNode(cache), key)
-                    || affinity.isBackup(localNode(cache), key)
-                    || nearEnabled())
-                    assertEquals(map.get(key), cache.localPeek(key, 
CachePeekMode.ONHEAP));
+                if (entry.primary() || entry.backup() || nearEnabled())
+                    assertEquals(map.get(key), cache.peek(key));
                 else
-                    assertNull(cache.localPeek(key, CachePeekMode.ONHEAP));
+                    assertNull(cache.peek(key));
             }
 
             emptyCaches.remove(cache);
         }
     }
+
+    /**
+     * Create list of keys for which the given cache is primary.
+     *
+     * @param cache Cache.
+     * @param cnt Keys count.
+     * @return Collection of keys for which given cache is primary.
+     */
+    private Iterable<Integer> 
primaryKeysForCache(CacheProjection<Integer,String> cache, int cnt) {
+        Collection<Integer> found = new ArrayList<>(cnt);
+
+        for (int i = 0; i < 10000; i++) {
+            if (cache.entry(i).primary()) {
+                found.add(i);
+
+                if (found.size() == cnt)
+                    return found;
+            }
+        }
+
+        throw new IllegalStateException("Unable to find " + cnt + " keys as 
primary for cache.");
+    }
 }

Reply via email to