svn commit: r1346896 - /openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java

[cu...@apache.org]

Author: curtisr7
Date: Wed Jun  6 14:00:05 2012
New Revision: 1346896

URL: http://svn.apache.org/viewvc?rev=1346896&view=rev
Log:
OPENJPA-2209: Add comment to AbstractBrokerFactory.

Modified:
    openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java

Modified: openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java?rev=1346896&r1=1346895&r2=1346896&view=diff
==============================================================================
--- openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java (original)
+++ openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java Wed Jun  6 14:00:05 2012
@@ -866,8 +866,12 @@ public abstract class AbstractBrokerFact
             newBroker(_conf.getConnectionUserName(), _conf.getConnectionPassword(),
                 _conf.isConnectionFactoryModeManaged(), _conf.getConnectionRetainModeConstant(), false).close();
         }
-        // Don't catch any exceptions here because we want to fail-fast if something bad happens when we're preloading.
+
+        // Don't get a MetaDataRepository yet if not preloading because it is possible that someone has extended the MDR
+        // and the extension hasn't been plugged in yet.
         if (MetaDataRepository.needsPreload(_conf) == true) {
+            // Don't catch any exceptions here because we want to fail-fast if something bad happens when we're
+            // preloading.
             MetaDataRepository mdr = _conf.getMetaDataRepositoryInstance();
             mdr.setValidate(MetaDataRepository.VALIDATE_RUNTIME, true);
             mdr.setResolve(MetaDataRepository.MODE_MAPPING_INIT, true);