james-server-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ian Boston (JIRA)" <server-...@james.apache.org>
Subject [jira] Commented: (IMAP-151) Replace MailboxMapper.existsMailboxStartingWith() with hasChildren()
Date Mon, 07 Jun 2010 16:31:46 GMT

    [ https://issues.apache.org/jira/browse/IMAP-151?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12876301#action_12876301
] 

Ian Boston commented on IMAP-151:
---------------------------------

Also there might be lots of other reasons there is a child node there.

There could be a 3rd party structure in the same space as the imap storage (this is unstructured
content so that is something that happens)

It would be safer to scan or query for jcr nodes that match a pattern (like have a mixin or
of a primary type), that really means they are imap folders.


> Replace MailboxMapper.existsMailboxStartingWith() with hasChildren()
> --------------------------------------------------------------------
>
>                 Key: IMAP-151
>                 URL: https://issues.apache.org/jira/browse/IMAP-151
>             Project: JAMES Imap
>          Issue Type: Improvement
>            Reporter: Tim-Christian Mundt
>         Attachments: IMAP-151.patch
>
>
> existsMailboxStartingWith() is only used to find out whether a mailbox has children.
For some implementations it's more efficient to use another mechanism (e.g. JCR nodes have
a hasNodes() method). Thus existsMailboxStartingWith() should be removed, hasChildren() should
be added and the implementations should make use of that.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org


Mime
View raw message