[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: backup5.patch

Backup5: 

clebert at clebert-laptop:/extra/clebert/workspaces/jboss-messaging-trunk/mina$ svn info
Path: .
URL: https://svn.jboss.org/repos/messaging/branches/Branch_JBMESSAGING-544
Repository Root: https://svn.jboss.org/repos/messaging
Repository UUID: b6fe21fe-e719-0410-baf4-a14e395396c4
Revision: 3435
Node Kind: directory
Schedule: normal
Last Changed Author: jmesnil
Last Changed Rev: 3431
Last Changed Date: 2007-12-06 10:38:05 -0600 (Thu, 06 Dec 2007)


D      src/main/org/jboss/jms/server/endpoint/advised
D      src/main/org/jboss/jms/server/endpoint/advised/ConnectionAdvised.java
D      src/main/org/jboss/jms/server/endpoint/advised/BrowserAdvised.java
D      src/main/org/jboss/jms/server/endpoint/advised/SessionAdvised.java
D      src/main/org/jboss/jms/server/endpoint/advised/ConnectionFactoryAdvised.java
D      src/main/org/jboss/jms/server/endpoint/advised/ConsumerAdvised.java
D      src/main/org/jboss/jms/server/endpoint/advised/AdvisedSupport.java
D      src/main/org/jboss/jms/server/container/ServerLogInterceptor.java
D      src/main/org/jboss/jms/client/container/ClientLogInterceptor.java
D      src/main/org/jboss/jms/client/container/ConnectionAspect.java
D      src/main/org/jboss/jms/client/container/JmsClientAspectXMLLoader.java
D      src/main/org/jboss/jms/client/container/StateCreationAspect.java
D      src/main/org/jboss/jms/client/container/ExceptionInterceptor.java
D      src/main/org/jboss/jms/client/container/ClusteringAspect.java
D      src/main/org/jboss/jms/client/container/BrowserAspect.java
D      src/main/org/jboss/jms/client/container/ProducerAspect.java
D      src/main/org/jboss/jms/client/container/SessionAspect.java
D      src/main/org/jboss/jms/client/container/DelegateIdentity.java
D      src/main/org/jboss/jms/client/container/ConsumerAspect.java
D      src/main/org/jboss/jms/client/ClientAOPStackLoader.java
D      src/etc/aop-messaging-client.xml
D      src/etc/aop-messaging-client-debug.xml
D      src/etc/prepare-aop.xml
D      src/etc/aop-messaging-server.xml
D      tests/src/org/jboss/test/messaging/tools/aop/PoisonInterceptor.java


> 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: backup2.patch, backup3.patch, backup5.patch, 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