james-server-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bernd Fondermann" <bernd.fonderm...@googlemail.com>
Subject Re: [MailboxAPI] Events
Date Sun, 18 Nov 2007 15:42:02 GMT
+1, sounds good, go ahead

  Bernd

On 11/18/07, Robert Burrell Donkin <robertburrelldonkin@gmail.com> wrote:
> unless anyone jumps in soon with objections or comments, i plan to
> commit changes that switch to using a session id and events
>
> On Nov 16, 2007 8:01 PM, Robert Burrell Donkin
> <robertburrelldonkin@gmail.com> wrote:
> > ATM the current design for MailboxListener (see 1) is method-oriented
> > - each event is reported by calling a specific method. extension
> > requires a new interface. i think that this API should be replaced by
> > one accepting an event. switching to a messaging API loosens coupling
> > and allows novel messages to be created without changing the
> > MailboxAPI.
> >
> > opinions?
> >
> > ATM mailbox events work by UidChangeTracker noticing that some data
> > that a mailbox has retrieved differs from it's cached value
> >
> > this approach causes problems for protocols such as IMAP that require
> > events generated by the current session to be handled differently from
> > events resulting from the actions of other sessions. IMO passing the
> > listener throw is a poor choice of design and a workaround for deeper
> > issues.
> >
> > IMO an explicit notion of MailboxAPI sessions is going to be required
> > (setting aside the current interface names). when a mailbox performs
> > an operation, it should notify listeners of that operation by posting
> > a suitable event containing a session id.
> >
> > opinions?
> >
> > - robert
> >
> > 1.
> http://svn.apache.org/repos/asf/james/server/trunk/core-library/src/main/java/org/apache/james/mailboxmanager/MailboxListener.java
> > 2.
> http://svn.apache.org/repos/asf/james/server/trunk/core-library/src/main/java/org/apache/james/mailboxmanager/tracking/UidChangeTracker.java
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
> For additional commands, e-mail: server-dev-help@james.apache.org
>
>

---------------------------------------------------------------------
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