svn commit: r945505 - in /james/server/trunk/pop3server/src: main/java/org/apache/james/pop3server/ main/java/org/apache/james/pop3server/core/ main/java/org/apache/james/pop3server/netty/ test/java/o

2010-05-17 Thread norman
Author: norman Date: Tue May 18 05:38:23 2010 New Revision: 945505 URL: http://svn.apache.org/viewvc?rev=945505&view=rev Log: Revert changes made for JAMES-1005. This was fixed in the Mailbox implementations, which is the right place. See IMAP-147 Modified: james/server/trunk/pop3server/src

svn commit: r945501 - /james/imap/trunk/mailbox/src/main/java/org/apache/james/imap/mailbox/Mailbox.java

2010-05-17 Thread norman
Author: norman Date: Tue May 18 05:21:16 2010 New Revision: 945501 URL: http://svn.apache.org/viewvc?rev=945501&view=rev Log: Just add a note in javadoc Modified: james/imap/trunk/mailbox/src/main/java/org/apache/james/imap/mailbox/Mailbox.java Modified: james/imap/trunk/mailbox/src/main/j

Re: Another OpenJPA InvalidStateException

2010-05-17 Thread Vanuj
Hi Norman, Verified it and the fix works. Thanks so much for that. But would it be a good idea to catch a generic Exception in ListCmdHandler and return error response (OR) wrap a generic exception into a MailboxException and throw it from StoreMailbox. Right now the client will hang every time

svn commit: r945365 - /james/imap/trunk/jpa/src/main/java/org/apache/james/imap/jpa/openjpa/OpenJPAMailbox.java

2010-05-17 Thread norman
Author: norman Date: Mon May 17 21:09:05 2010 New Revision: 945365 URL: http://svn.apache.org/viewvc?rev=945365&view=rev Log: Remove unused import Modified: james/imap/trunk/jpa/src/main/java/org/apache/james/imap/jpa/openjpa/OpenJPAMailbox.java Modified: james/imap/trunk/jpa/src/main/java

[jira] Resolved: (IMAP-147) Mailbox instances should be valid within many requests.

2010-05-17 Thread Norman Maurer (JIRA)
[ https://issues.apache.org/jira/browse/IMAP-147?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Norman Maurer resolved IMAP-147. Resolution: Fixed Fixed.. > Mailbox instances should be valid within many requests. > --

svn commit: r945363 - in /james/imap/trunk: jcr/src/main/java/org/apache/james/imap/jcr/ jpa/src/main/java/org/apache/james/imap/jpa/ jpa/src/main/java/org/apache/james/imap/jpa/openjpa/

2010-05-17 Thread norman
Author: norman Date: Mon May 17 21:06:16 2010 New Revision: 945363 URL: http://svn.apache.org/viewvc?rev=945363&view=rev Log: Mailbox instances are valid within many requests now. (IMAP-147) Modified: james/imap/trunk/jcr/src/main/java/org/apache/james/imap/jcr/JCRMailbox.java james/imap

[jira] Created: (IMAP-147) Mailbox instances should be valid within many requests.

2010-05-17 Thread Norman Maurer (JIRA)
Mailbox instances should be valid within many requests. --- Key: IMAP-147 URL: https://issues.apache.org/jira/browse/IMAP-147 Project: JAMES Imap Issue Type: Bug Components: JCR M

[jira] Assigned: (JAMES-1003) ClassLoader with External libs

2010-05-17 Thread Norman Maurer (JIRA)
[ https://issues.apache.org/jira/browse/JAMES-1003?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Norman Maurer reassigned JAMES-1003: Assignee: Norman Maurer > ClassLoader with External libs > -- >

Re: Another OpenJPA InvalidStateException

2010-05-17 Thread Norman Maurer
Hi Vanuj, I think I fixed the bug. Please update you source tree and build it again. Wait for your feedback Thx, Norman 2010/5/17 Norman Maurer : > Hi Vanuj, > > I found the problem with pop3. At least I know why it happens. But I > have to think a bit about how to fix it. Fix will come so

[jira] Resolved: (JAMES-1006) LIST msgnum fails with "-ERR Message (1) already deleted."

2010-05-17 Thread Norman Maurer (JIRA)
[ https://issues.apache.org/jira/browse/JAMES-1006?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Norman Maurer resolved JAMES-1006. -- Resolution: Fixed fixed > LIST msgnum fails with "-ERR Message (1) already deleted." > ---

svn commit: r945359 - /james/server/trunk/pop3server/src/main/java/org/apache/james/pop3server/core/ListCmdHandler.java

2010-05-17 Thread norman
Author: norman Date: Mon May 17 20:44:30 2010 New Revision: 945359 URL: http://svn.apache.org/viewvc?rev=945359&view=rev Log: Fix LIST msgnum command (JAMES-1006) Modified: james/server/trunk/pop3server/src/main/java/org/apache/james/pop3server/core/ListCmdHandler.java Modified: james/serv

[jira] Created: (JAMES-1006) LIST msgnum fails with "-ERR Message (1) already deleted."

2010-05-17 Thread Norman Maurer (JIRA)
LIST msgnum fails with "-ERR Message (1) already deleted." -- Key: JAMES-1006 URL: https://issues.apache.org/jira/browse/JAMES-1006 Project: JAMES Server Issue Type: Bug Compo

[jira] Resolved: (JAMES-1005) POP3 Stat/List hangs on second execution during POP3Session

2010-05-17 Thread Norman Maurer (JIRA)
[ https://issues.apache.org/jira/browse/JAMES-1005?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Norman Maurer resolved JAMES-1005. -- Resolution: Fixed Fixed. Thx for reporting > POP3 Stat/List hangs on second execution during P

svn commit: r945355 - in /james/server/trunk/pop3server/src: main/java/org/apache/james/pop3server/ main/java/org/apache/james/pop3server/core/ main/java/org/apache/james/pop3server/netty/ test/java/o

2010-05-17 Thread norman
Author: norman Date: Mon May 17 20:37:00 2010 New Revision: 945355 URL: http://svn.apache.org/viewvc?rev=945355&view=rev Log: Mailbox instances should only be used within a request. Be sure we use a "new instance" on each pop3 command (JAMES-1005) Modified: james/server/trunk/pop3server/src

[jira] Created: (JAMES-1005) POP3 Stat/List hangs on second execution during POP3Session

2010-05-17 Thread Norman Maurer (JIRA)
POP3 Stat/List hangs on second execution during POP3Session --- Key: JAMES-1005 URL: https://issues.apache.org/jira/browse/JAMES-1005 Project: JAMES Server Issue Type: Bug Affects V

[jira] Resolved: (JAMES-1004) persistence.xml use jpa spec 1.0 but should use 2.0

2010-05-17 Thread Norman Maurer (JIRA)
[ https://issues.apache.org/jira/browse/JAMES-1004?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Norman Maurer resolved JAMES-1004. -- Resolution: Fixed fixed! > persistence.xml use jpa spec 1.0 but should use 2.0 > -

Re: Another OpenJPA InvalidStateException

2010-05-17 Thread Norman Maurer
Hi Vanuj, I found the problem with pop3. At least I know why it happens. But I have to think a bit about how to fix it. Fix will come soon. Thx, Norman 2010/5/17 Norman Maurer : > Hi Vanuj, > > The error below is fixed: > > 312  James  TRACE  [main] openjpa.Runtime - Not creating a > ValidatorI

Re: Uid Generation Strategy

2010-05-17 Thread Eric Charles
Hi Norman, As discussed, it may be worth to reread the RFC requirements: I've copy/paste the RFC paragraph here after. Can we deduce from that that the uid must be incremented by one for each mail arriving in a mailbox? Tks, Eric 2.3.1.1.Unique Identifier (UID) Message Attribute A

Re: Another OpenJPA InvalidStateException

2010-05-17 Thread Norman Maurer
Hi Vanuj, The error below is fixed: 312 James TRACE [main] openjpa.Runtime - Not creating a ValidatorImpl because this app is using the JPA 1.0 Spec The error message about Weaver can be ignored: > Caused by: java.lang.IllegalStateException: Cannot apply class transformer > without LoadTime

[jira] Created: (JAMES-1004) persistence.xml use jpa spec 1.0 but should use 2.0

2010-05-17 Thread Norman Maurer (JIRA)
persistence.xml use jpa spec 1.0 but should use 2.0 --- Key: JAMES-1004 URL: https://issues.apache.org/jira/browse/JAMES-1004 Project: JAMES Server Issue Type: Bug Affects Versions: 3.0, 3.

svn commit: r945321 - /james/server/trunk/spring-deployment/src/main/config/james/META-INF/persistence.xml

2010-05-17 Thread norman
Author: norman Date: Mon May 17 19:23:53 2010 New Revision: 945321 URL: http://svn.apache.org/viewvc?rev=945321&view=rev Log: use jpa spec version 2.0 in persistence.xml (JAMES-1004) Modified: james/server/trunk/spring-deployment/src/main/config/james/META-INF/persistence.xml Modified: jam

Re: Another OpenJPA InvalidStateException

2010-05-17 Thread Eric Charles
Hi Vanuj, Seems like your eclipse has some confusion with openjpa1 and openjpa2. (James uses openjpa2 since 1 week). I also activated the TRACE log, and didn't receive your message "[main] openjpa.Runtime - Not creating a ValidatorImpl because this app is using the JPA 1.0 Spec " Both exceptio

[jira] Updated: (JAMES-1003) ClassLoader with External libs

2010-05-17 Thread Eric Charles (JIRA)
[ https://issues.apache.org/jira/browse/JAMES-1003?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Eric Charles updated JAMES-1003: Attachment: jamesclassloader.patch Patch proposal is attached. > ClassLoader with External libs >

[jira] Created: (JAMES-1003) ClassLoader with External libs

2010-05-17 Thread Eric Charles (JIRA)
ClassLoader with External libs -- Key: JAMES-1003 URL: https://issues.apache.org/jira/browse/JAMES-1003 Project: JAMES Server Issue Type: New Feature Components: Deployment Modules Affects Versions:

Re: Another OpenJPA InvalidStateException

2010-05-17 Thread Apache James
Hi Norman, I enabled trace and found that there are two exceptions happening. One on james startup and the other when the LIST or RETR command gets called. Only the one on startup gets logged. Also, There is this message during printed during server startup that looked interesting. (Is is sup

Re: Another OpenJPA InvalidStateException

2010-05-17 Thread Eric Charles
Similar issues on POP3 the last weeks with JPA exceptions not logged in james-server.log. I have now configured openjpa.Log in persistence.xml and added the following line in log4j.properties log4j.logger.org.apache.openjpa=TRACE, CONS, FILE But I still don't see much openjpa in the logs. Er