Fw: svn commit: r325989 - /james/server/trunk/src/java/org/apache/james/mailrepository/AvalonMailRepository.java

[Guru Shenoy]

sorry for providing this in installments. But looks like the

JDBCMailRepository needs this fix too..here is the svn diff. I dont have a setup to test the JDBCMailRepository fix here.

+++ JDBCMailRepository.java     (working copy)
@@ -493,9 +493,7 @@

.append(new java.util.Date(System.currentTimeMillis()));

                getLogger().debug(debugBuffer.toString());
            }
-            synchronized (this) {
-                notify();
-            }
            return true;
        } else {
            return false;
@@ -772,6 +770,9 @@
            if (!wasLocked) {
                // If it wasn't locked, we need to unlock now
                unlock(key);
+                synchronized (this) {
+                    notify();
+                }
            }
        }
    }

----- Original Message ----- From: <[EMAIL PROTECTED]>

To: <[EMAIL PROTECTED]>
Sent: Monday, October 17, 2005 2:52 PM

Subject: svn commit: r325989 - /james/server/trunk/src/java/org/apache/james/mailrepository/AvalonMailRepository.java

Author: noel
Date: Mon Oct 17 14:52:25 2005
New Revision: 325989

URL: http://svn.apache.org/viewcvs?rev=325989&view=rev
Log:

Applied Guru Shenoy's patch to fix CPU utilization problem with multiple delivery threads.

Modified:

james/server/trunk/src/java/org/apache/james/mailrepository/AvalonMailRepository.java

Modified: james/server/trunk/src/java/org/apache/james/mailrepository/AvalonMailRepository.java URL: http://svn.apache.org/viewcvs/james/server/trunk/src/java/org/apache/james/mailrepository/AvalonMailRepository.java?rev=325989&r1=325988&r2=325989&view=diff

==============================================================================

--- james/server/trunk/src/java/org/apache/james/mailrepository/AvalonMailRepository.java (original) +++ james/server/trunk/src/java/org/apache/james/mailrepository/AvalonMailRepository.java Mon Oct 17 14:52:25 2005

@@ -200,9 +200,6 @@

.append(new java.util.Date(System.currentTimeMillis()));

                getLogger().debug(debugBuffer.toString());
            }
-            synchronized (this) {
-                notify();
-            }
            return true;
        } else {
            return false;
@@ -303,6 +300,9 @@
                if (!wasLocked) {
                    // If it wasn't locked, we need to unlock now
                    unlock(key);
+                    synchronized (this) {
+                        notify();
+                    }
                }
            }




---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]