Author: norman
Date: Mon Mar 29 17:59:39 2010
New Revision: 928850
URL: http://svn.apache.org/viewvc?rev=928850&view=rev
Log:
Dispose source mail after copy it. This is prolly the cause of OOM reported by
Eric. Thx again for the report
Modified:
james/server/trunk/spoolmanager/src/main/java/org/apache/james/AbstractMailServer.java
Modified:
james/server/trunk/spoolmanager/src/main/java/org/apache/james/AbstractMailServer.java
URL:
http://svn.apache.org/viewvc/james/server/trunk/spoolmanager/src/main/java/org/apache/james/AbstractMailServer.java?rev=928850&r1=928849&r2=928850&view=diff
==============================================================================
---
james/server/trunk/spoolmanager/src/main/java/org/apache/james/AbstractMailServer.java
(original)
+++
james/server/trunk/spoolmanager/src/main/java/org/apache/james/AbstractMailServer.java
Mon Mar 29 17:59:39 2010
@@ -52,6 +52,7 @@ import org.apache.james.core.MailHeaders
import org.apache.james.core.MailImpl;
import org.apache.james.impl.jamesuser.JamesUsersRepository;
import org.apache.james.lifecycle.Configurable;
+import org.apache.james.lifecycle.LifecycleUtil;
import org.apache.james.lifecycle.LogEnabled;
import org.apache.james.services.MailServer;
import org.apache.james.transport.camel.InMemoryMail;
@@ -298,7 +299,10 @@ public abstract class AbstractMailServer
*/
public void sendMail(Mail mail) throws MessagingException {
try {
- producerTemplate.sendBody(getToUri(mail), ExchangePattern.InOnly,
new InMemoryMail(mail));
+ Mail newMail = new InMemoryMail(mail);
+ producerTemplate.sendBody(getToUri(mail), ExchangePattern.InOnly,
newMail);
+
+ LifecycleUtil.dispose(mail);
} catch (Exception e) {
logger.error("Error storing message: " + e.getMessage(),e);
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
