Re: Patch for org.apache.jdo.tck.api.persistencemanager.cache.Retrieve

[Michelle Caisse <Mi...@Sun.COM>]

Hi Erik,

This patch looks good.  The test passes with it.  Please go ahead and 
check it in.

Thanks,
Michelle

erik@jpox.org wrote:

>Hi,
>
>
>Here is a patch for tests that asserts PM.Retrieve operations not using the
>fetch plan.
>
>
>
>Index:
>C:/jdo/root/tck20/test/java/org/apache/jdo/tck/api/persistencemanager/cache/Retrieve.java
>===================================================================
>---
>C:/jdo/root/tck20/test/java/org/apache/jdo/tck/api/persistencemanager/cache/Retrieve.java	(revision
>291420)
>+++
>C:/jdo/root/tck20/test/java/org/apache/jdo/tck/api/persistencemanager/cache/Retrieve.java	(working
>copy)
>@@ -173,7 +173,7 @@
>             coll.add(p1);
>             coll.add(p3);
>             coll.add(rect);
>-            pm.retrieveAll(coll, true);
>+            pm.retrieveAll(coll, false);
>             pm.makeTransientAll(coll);
>             tx.commit();
>             tx = null;
>@@ -248,7 +248,7 @@
>             objs[0] = p1;
>             objs[1] = p3;
>             objs[2] = rect;
>-            pm.retrieveAll(objs, true);
>+            pm.retrieveAll(objs, false);
>             pm.makeTransientAll(objs);
>             tx.commit();
>             tx = null;
>
>  
>