Doing the mediator API related changes I asked myself whether concrete mediators like FaultMediator should really widen the visibility of AbstractMediatorFactory#createSpecificMediator. At least some of Synapse built-in mediators change the visibility from protected to public. I did not check all of them. I don’t think this is generally a good idea as to my understanding “users” of any MediatorFactory implementation should always only call createMediator (which internally delegates to createSpecificMediator). Otherwise functionality from the AbstractMediatorFactory might be mistakenly bypassed.
Another small side-note AbstractMediatorFactory#createMediator’s JavaDoc has not been adjusted to the added parameter “Properties” which use is still unclear to me. Apart from that I have positive feedback regarding my tests with simple mediation and usage of the nhttp transports. I could not spot any problem so far. I also did a couple of small load tests against one small instance reaching about 2500 requests per second and letting it run on that level for a few hours. Regards, Eric ________________________________ From: Ruwan Linton [mailto:ruwan.lin...@gmail.com] Sent: Sunday, December 12, 2010 3:33 AM To: dev@synapse.apache.org Subject: Re: [VOTE] Release Synapse-2.0.0 (Take2) Hi Eric, On Sat, Dec 11, 2010 at 9:40 PM, Hubert, Eric <eric.hub...@foxmobile.com<mailto:eric.hub...@foxmobile.com>> wrote: I found another issue with the nhttp transports: I guess Synapse 2.0 release should not be shipped with http core / http core nio 4.1 alpha 1 dependencies, but the final release version 4.1. I also think the patch for httpcore-193 is no longer needed and the binary patch should not be in the patches directory. Cool, I guess a good catch, will fix it. Apart from that I finished the mediator migration and I’m now starting to test including custom mediators. Unfortunately I will have to continue tomorrow, as I have to take care of other things today. Sure no problem. Regards, Eric PS: Is there already an updated version of the migration tool to test? Unfortunately not yet Eric, I didn't have time to work on that, planning to work on resolving all the issue today starting from now on. :-) Will post the tool once there is an update. Thanks, Ruwan ________________________________ From: Ruwan Linton [mailto:ruwan.lin...@gmail.com<mailto:ruwan.lin...@gmail.com>] Sent: Thursday, December 09, 2010 11:08 AM To: dev@synapse.apache.org<mailto:dev@synapse.apache.org> Subject: Re: [VOTE] Release Synapse-2.0.0 (Take2) Hiranya, I've fixed this dependency issue, I expect you to do a build on the local machine and check this, will wait for your comments to the next RC :-) Thanks, Ruwan On Thu, Dec 9, 2010 at 12:44 AM, Hiranya Jayathilaka <hiranya...@gmail.com<mailto:hiranya...@gmail.com>> wrote: I hate to be a PITA but I see another issue with this. The FIX transport has been included in the latest binary. So that's good. But I can see that Quickfix/J has also crept into the distro (about 9MB). FIX transport also requires MINA and SLF4J which are not included in the binary. So IMO either we should include all the required dependencies or none of them. I think we should add an exclusion to Quickfix/J and keep it out of the binary. WDYT? Thanks, Hiranya On Wed, Dec 8, 2010 at 12:27 PM, Ruwan Linton <ruwan.lin...@gmail.com<mailto:ruwan.lin...@gmail.com>> wrote: > Hi Devs, > > This is take 2 call for votes to release Synapse-2.0.0. > > Please review the signed artifacts: > http://people.apache.org/~ruwan/synapse/2.0.0-RC2/artifacts/ > The m2 repository is available at: > http://people.apache.org/~ruwan/synapse/2.0.0-RC2/m2_repo/ > Site update for this release is available at: > http://people.apache.org/~ruwan/synapse/2.0.0-RC2/site/ > SVN Info: > revision is 1043322 on > https://svn.apache.org/repos/asf/synapse/branches/2.0 > Here's my +1 to declaring the above dist as Synapse-2.0.0. > > Thanks, > Ruwan > PS: The KEYS file for signing these > releases http://www.apache.org/dist/synapse/KEYS > -- > Ruwan Linton > Software Architect & Product Manager, WSO2 ESB; http://wso2.org/esb > WSO2 Inc.; http://wso2.org > > Lean . Enterprise . Middleware > > phone: +1 408 754 7388 ext 51789 > email: ru...@wso2.com<mailto:ru...@wso2.com>; cell: +94 77 341 3097 > blog: http://blog.ruwan.org > linkedin: http://www.linkedin.com/in/ruwanlinton > google: http://www.google.com/profiles/ruwan.linton > tweet: http://twitter.com/ruwanlinton > -- Hiranya Jayathilaka Senior Software Engineer; WSO2 Inc.; http://wso2.org E-mail: hira...@wso2.com<mailto:hira...@wso2.com>; Mobile: +94 77 633 3491 Blog: http://techfeast-hiranya.blogspot.com --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@synapse.apache.org<mailto:dev-unsubscr...@synapse.apache.org> For additional commands, e-mail: dev-h...@synapse.apache.org<mailto:dev-h...@synapse.apache.org> -- Ruwan Linton Software Architect & Product Manager, WSO2 ESB; http://wso2.org/esb WSO2 Inc.; http://wso2.org Lean . Enterprise . Middleware phone: +1 408 754 7388 ext 51789 email: ru...@wso2.com<mailto:ru...@wso2.com>; cell: +94 77 341 3097 blog: http://blog.ruwan.org linkedin: http://www.linkedin.com/in/ruwanlinton google: http://www.google.com/profiles/ruwan.linton tweet: http://twitter.com/ruwanlinton -- Ruwan Linton Software Architect & Product Manager, WSO2 ESB; http://wso2.org/esb WSO2 Inc.; http://wso2.org Lean . Enterprise . Middleware phone: +1 408 754 7388 ext 51789 email: ru...@wso2.com<mailto:ru...@wso2.com>; cell: +94 77 341 3097 blog: http://blog.ruwan.org linkedin: http://www.linkedin.com/in/ruwanlinton google: http://www.google.com/profiles/ruwan.linton tweet: http://twitter.com/ruwanlinton