Re: svn commit: r513183 - in /incubator/openjpa/trunk: openjpa-kernel/src/main/java/org/apache/openjpa/conf/ openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ openjpa-persistence/src/main/java/org/apache/openjpa/persistence/

[Craig L Russell]

Hi Patrick,




Craig

On Feb 28, 2007, at 10:31 PM, [EMAIL PROTECTED] wrote:

I don't quite understand the code that follows. What is the difference between this code and simply defaulting the value to non- finalizing impl?

         PersistenceUnitInfo pui, Map m) {

         try {
+            // if the BrokerImpl hasn't been specified, switch to the

+            // should be doing proper resource management.
+            if (!Configurations.containsProperty(BrokerValue.KEY,
+                pui.getProperties())

+            }
+
             ConfigurationProvider cp = pd.load(pui, m);
             if (cp == null)
                 return null;

PersistenceProductDerivation pd = new PersistenceProductDerivation(); + // non-finalizing one, since anything claiming to be a container + && !Configurations.containsProperty (BrokerValue.KEY, m)) { + m.put(BrokerValue.KEY, BrokerValue.NON_FINALIZING_ALIAS);

Craig Russell
Architect, Sun Java Enterprise System http://java.sun.com/products/jdo
408 276-5638 mailto:[EMAIL PROTECTED]
P.S. A good JDO? O, Gasp!

smime.p7s
Description: S/MIME cryptographic signature