[jboss-jira] [JBoss JIRA] Updated: (JBMESSAGING-681) Refactor client side model

[Clebert Suconic (JIRA)]

     [ http://jira.jboss.com/jira/browse/JBMESSAGING-681?page=all ]

Clebert Suconic updated JBMESSAGING-681:
----------------------------------------

    Attachment: patch.diff

Backup point as I don't have where to commit this yet.

patch applied on Jeff's branch, version 3385

This is just a simple refactoring where the new model isn't being target yet.

On this patch, you would still find ClosedInterpcetor, FailoverValveinterceptor, ConsumerAspect and ClusteringAspect... all other aspects would have to be removed by hand. (as the patch won't remove them).

> Refactor client side model
> --------------------------
>
>                 Key: JBMESSAGING-681
>                 URL: http://jira.jboss.com/jira/browse/JBMESSAGING-681
>             Project: JBoss Messaging
>          Issue Type: Task
>            Reporter: Tim Fox
>         Assigned To: Clebert Suconic
>            Priority: Critical
>             Fix For: 2.0.0 Alpha
>
>         Attachments: patch.diff
>
>
> Currently, client side JMS functionality is implemented in a set of aspects: SessionAspect, TransactionAspect, FactoryAspect, etc. There seems no good reason to have these as separate aspects since we either provide all the jms functionality or none of it.
> Having the functionality distributed across several aspects makes it hard to communicate between them meaning the code is messy and hard to follow.
> We should consolidate into one aspect, from which hang a set of classes Session, Connection, Consumer etc which implement the jms functionality in a nice OO way. This will make the code simpler and easier to understand and possibly more performant since we would avoid unnecessary calls through the stack.
> The classes should be chosen to represent generic messaging, not just JMS (e.g. no need for a producer).
> We also need to have an extension point so customers can add their own vanilla interceptors.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira