[prev in list] [next in list] [prev in thread] [next in thread] 

List:       synapse-dev
Subject:    Re: Eventing code review
From:       Asanka Abeysinghe <asankaa () wso2 ! com>
Date:       2009-01-10 18:55:10
Message-ID: 4968EC3E.5000900 () wso2 ! com
[Download RAW message or body]

Hi Asankha,
Thanks for the review, will refactor the package to fulfill [1].
Thx
Asanka
 
Asankha C. Perera wrote:
> Hi Asanka
>
> It seems like most of the code in the eventing package exceeds the 100 
> line limit.. and lacking very important Javadocs.. e.g. 
> SynapseSubscriptionManager, SynapseSubscription, SynapseEventSource, 
> SynapseEventFilter
>
> Could you revisit the package and make the necessary changes, as per 
> [1]. Also since this package has a core dependency with the interfaces 
> from org.wso2.eventing, could you extract the core documentation (say 
> for Subscription etc) into a package file
>
> thanks
> asankha
>
> [1] http://wiki.apache.org/ws/Synapse/Developers
>


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

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic