Fwd: svn commit: r612846 - in /openjpa: branches/1.0.x/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java

[Craig L Russell]

Can someone in infra take a look at why the JIRA is not getting updated with the svn commit data?

Thanks,

Craig

Begin forwarded message:

From: Kevin Sutter <[EMAIL PROTECTED]>
Date: January 18, 2008 7:13:13 AM PST
To: [EMAIL PROTECTED]

Reply-To: dev@openjpa.apache.org

Hi,


Kevin

On Jan 17, 2008 9:35 AM, <[EMAIL PROTECTED]> wrote:

Subject: Re: svn commit: r612846 - in /openjpa: branches/1.0.x/ openjpa-kernel/src/main/java/org/apache/openjpa/kernel/ AbstractBrokerFactory.java trunk/openjpa-kernel/src/main/java/org/ apache/openjpa/kernel/AbstractBrokerFactory.java Is there a problem with getting the SVN commits into the JIRA issues? I thought I had noticed this problem a week or so ago, but then forgot to follow up. I am now noticing it again with my latest commit. I committed these changes early yesterday morning and they are still not attached to the JIRA reports. I thought waiting 24 hours would be sufficient... Any ideas?

Author: kwsutter
Date: Thu Jan 17 07:35:45 2008
New Revision: 612846

URL: http://svn.apache.org/viewvc?rev=612846&view=rev
Log:
OPENJPA-437.  Changed AbstractBrokerFactory.loadPersistentTypes to be




Modified:





Modified:

URL:
http://svn.apache.org/viewvc/openjpa/branches/1.0.x/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java?rev=612846&r1=612845&r2=612846&view=diff


---

(original)
+++

Thu Jan 17 07:35:45 2008
@@ -133,6 +133,7 @@
    */
   protected AbstractBrokerFactory(OpenJPAConfiguration config) {
       _conf = config;

   }

   /**
@@ -231,8 +232,13 @@
   /**
    * Load the configured persistent classes list. Performed
automatically
    * whenever a broker is created.
+     *

brokers

structures

thread
+     * safe and this was an easy, efficient solution (OPENJPA-437).
    */
-    private void loadPersistentTypes(ClassLoader envLoader) {

{
       // no listed persistent types?
       if (_pcClassNames != null && _pcClassNames.isEmpty())
           return;
@@ -245,7 +251,7 @@
           Collection clss = _conf.getMetaDataRepositoryInstance().
               loadPersistentTypes(false, loader);
           if (clss.isEmpty())
-                _pcClassNames = Collections.EMPTY_SET;
+                _pcClassNames = Collections.EMPTY_LIST;
           else {
               _pcClassNames = new ArrayList(clss.size());
               for (Iterator itr = clss.iterator(); itr.hasNext();) {
@@ -254,7 +260,6 @@
                   if (needsSub(cls))
                       toRedefine.add(cls);
               }
-                _pcClassLoaders = new ReferenceHashSet(
ReferenceHashSet.WEAK);
               _pcClassLoaders.add(loader);
           }
       } else {

Modified:

URL:
http://svn.apache.org/viewvc/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/AbstractBrokerFactory.java?rev=612846&r1=612845&r2=612846&view=diff


---

(original)
+++

Thu Jan 17 07:35:45 2008
@@ -256,8 +256,13 @@
   /**
    * Load the configured persistent classes list. Performed
automatically
    * whenever a broker is created.
+     *

brokers

structures

thread
+     * safe and this was an easy, efficient solution (OPENJPA-437).
    */
-    private void loadPersistentTypes(ClassLoader envLoader) {

{
       // no listed persistent types?
       if (_pcClassNames != null && _pcClassNames.isEmpty())
           return;
@@ -270,7 +275,7 @@
           Collection clss = _conf.getMetaDataRepositoryInstance().
               loadPersistentTypes(false, loader);
           if (clss.isEmpty())
-                _pcClassNames = Collections.EMPTY_SET;
+                _pcClassNames = Collections.EMPTY_LIST;
           else {
               _pcClassNames = new ArrayList(clss.size());
               for (Iterator itr = clss.iterator(); itr.hasNext();) {

synchronized to get around the multi-threading issue during broker creation. Also, migrated the change for OPENJPA-449 from trunk to the 1.0.x branch. openjpa/branches/1.0.x/openjpa-kernel/src/main/java/org/apache/ openjpa/kernel/AbstractBrokerFactory.java openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ kernel/AbstractBrokerFactory.java openjpa/branches/1.0.x/openjpa-kernel/src/main/java/org/apache/ openjpa/kernel/AbstractBrokerFactory.java = = = = = = = = = ===================================================================== openjpa/branches/1.0.x/openjpa-kernel/src/main/java/org/apache/ openjpa/kernel/AbstractBrokerFactory.java openjpa/branches/1.0.x/openjpa-kernel/src/main/java/org/apache/ openjpa/kernel/AbstractBrokerFactory.java + _pcClassLoaders = new ReferenceHashSet(ReferenceHashSet.WEAK); + * This method is synchronized due to the possible creation of new + * (entity managers) by multiple threads (clients). The two data + * used by this method (_pcClassNames and _pcClassLoaders) are not + private synchronized void loadPersistentTypes(ClassLoader envLoader) openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ kernel/AbstractBrokerFactory.java = = = = = = = = = ===================================================================== openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ kernel/AbstractBrokerFactory.java openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/ kernel/AbstractBrokerFactory.java + * This method is synchronized due to the possible creation of new + * (entity managers) by multiple threads (clients). The two data + * used by this method (_pcClassNames and _pcClassLoaders) are not + private synchronized void loadPersistentTypes(ClassLoader envLoader)

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