svn commit: r957505 - /james/imap/trunk/jcr/src/main/java/org/apache/james/imap/jcr/mail/model/JCRMessage.java

2010-06-24 Thread norman
Author: norman Date: Thu Jun 24 10:55:49 2010 New Revision: 957505 URL: http://svn.apache.org/viewvc?rev=957505view=rev Log: Only set the message flags when save a already existing message. This prevent JCR for use to much memory when update messages an improve the performance a lot (IMAP-159)

[jira] Resolved: (IMAP-159) OOM when using JCR when select folder with many recent messages

2010-06-24 Thread Norman Maurer (JIRA)
[ https://issues.apache.org/jira/browse/IMAP-159?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Norman Maurer resolved IMAP-159. Resolution: Fixed fixed.. OOM when using JCR when select folder with many recent messages

[jira] Resolved: (IMAP-162) Mailbox rename need to rename the node too

2010-06-24 Thread Norman Maurer (JIRA)
[ https://issues.apache.org/jira/browse/IMAP-162?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Norman Maurer resolved IMAP-162. Resolution: Fixed Mailbox rename need to rename the node too

svn commit: r957649 - in /james/imap/trunk/jcr/src/main/java/org/apache/james/imap/jcr: mail/JCRMailboxMapper.java mail/JCRMessageMapper.java user/JCRSubscriptionMapper.java

2010-06-24 Thread norman
Author: norman Date: Thu Jun 24 17:42:31 2010 New Revision: 957649 URL: http://svn.apache.org/viewvc?rev=957649view=rev Log: more XPATH query tuning Modified: james/imap/trunk/jcr/src/main/java/org/apache/james/imap/jcr/mail/JCRMailboxMapper.java

Re: OpenJPA usage at Apache James

2010-06-24 Thread Eric Charles
Hi Pinaki, OpenJPA 2.1.0-SNAPSHOT didn't help. I also made a separate sample with a Message, AbstractMessage and Header (a message has many headers). I defined: - @OneToMany(mappedBy...) with type Header on the AbstractMessage - @ManyToOne with type AbstractMessage on the Header. With that

JPA for imap 0.1 release

2010-06-24 Thread Eric Charles
Hi, Before releasing IMAP, I propose the following for JPA: 1. JAMES ships with openjpa but is open to other JPA providers via the provider/ tag: - org.apache.openjpa.persistence.PersistenceProviderImpl for openjpa (the standard) - org.hibernate.ejb.HibernatePersistence for hibernate -

Re: JPA for imap 0.1 release

2010-06-24 Thread Tim-Christian Mundt
Hi, 4. The majority of the classes will use openjpa classes: instead of moving them all to org.apache.james.imap.jpa.mail.model.openjpa package, we leave them and we rename this package to org.apache.james.imap.jpa.mail.model.streaming to reflect that it goes on streaming the blobs. With

[IMAP] countMailboxesWithName

2010-06-24 Thread Tim-Christian Mundt
Hi, in IMAP the method MailboxMapper.countMailboxesWithName(mailboxName) is only ever used in StoreMailboxManager.mailboxExists(mailboxName, session). There the result is checked for duplicates. Duplicates should be detected before writing and not when reading, for some stores (e.g. maildir) it

Re: [IMAP] countMailboxesWithName

2010-06-24 Thread Eric Charles
oops, I didn't mean so patch it but so, patches would be good for those kind of behaviour. Still early and missing coffee :) Tks, Eric On 06/25/2010 05:29 AM, Eric Charles wrote: Hi Tim, Same for the domain creation. Currently, you may have duplicates in db, and when you list domains, it

Re: [IMAP] countMailboxesWithName

2010-06-24 Thread Eric Charles
Hi Tim, Same for the domain creation. Currently, you may have duplicates in db, and when you list domains, it hides it via a Set. A strategy is to check if it already exists before writing it. So patch it :) Tks, Eric On 06/24/2010 10:41 PM, Tim-Christian Mundt wrote: Hi, in IMAP the

Re: JPA for imap 0.1 release

2010-06-24 Thread Eric Charles
Hi Tim, If we set streaming by default, we can not use derby anymore as default. With a different provider, schema may be different. Do you mean we should ensure with specific annotations that schema will always be the same? Tks, Eric On 06/24/2010 10:31 PM, Tim-Christian Mundt wrote: