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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to