Hi, On Fri, Apr 20, 2012 at 9:07 AM, Anjana Fernando <[email protected]> wrote:
> Hi Lakmali, > > 6 TPS for a small message is very unlikely! .. did you by any chance put a > time interval between tests in SOAPUI, if so, make it 0 and try the tests > again. > Yes, the Test Delay value had been set to 1000. Thanks for pointing it. Done the tests again by making it to 0 and updated the doc [1]. [1] https://docs.google.com/a/wso2.com/document/d/1N2uAxA0G-zmGFgSP6K-AVNtHZVZBZxvIrOiJwu57Ds4/edit Thanks, Lakmali > > Cheers, > Anjana. > > > On Fri, Apr 20, 2012 at 9:04 AM, Lakmali Baminiwatta <[email protected]>wrote: > >> Hi, >> >> >> On Fri, Apr 20, 2012 at 8:44 AM, Anjana Fernando <[email protected]> wrote: >> >>> Hi Lakmali, >>> >>> Good work! .. Also for the performance comparison, I guess you've used >>> only big messages to test it, you should also test it with smaller messages >>> to get a much higher TPS to check whether other parts are not affected. >>> >> >> Actually the message used in the Smooks transformation is a smaller >> message which has only 20 result elements. Since I ran the test (Test-1) >> using 5 threads within 60 seconds the bytes processed indicates a larger >> value. I think I should add the message used for the performance test to >> the doc. >> >> Thanks, >> Lakmali >> >>> >>> Cheers, >>> Anjana. >>> >>> On Thu, Apr 19, 2012 at 5:48 PM, Lakmali Baminiwatta >>> <[email protected]>wrote: >>> >>>> Hi, >>>> >>>> A performance comparison was done between the improved smooks mediator >>>> and the current smooks mediator. According to the test results there was no >>>> considerable difference in performance between two mediators. I have shared >>>> a doc [1] with the complete performance test information with results. >>>> >>>> Further changes were done to the code as discussed in the review. >>>> >>>> [1] >>>> https://docs.google.com/a/wso2.com/document/d/1N2uAxA0G-zmGFgSP6K-AVNtHZVZBZxvIrOiJwu57Ds4/edit >>>> >>>> Thanks, >>>> Lakmali >>>> >>>> >>>> On Thu, Apr 19, 2012 at 5:47 PM, Lakmali Baminiwatta >>>> <[email protected]>wrote: >>>> >>>>> Hi, >>>>> >>>>> Here are the review notes >>>>> >>>>> >>>>> 1. Allow to result a Java output even when Smooks is using JPA >>>>> persistence execution. >>>>> 2. Method name resizeBuffer() in ElementOutputStream class is >>>>> confusing. Therefore add a more clarified comment. >>>>> 3. Streaming code is complex, hence add more comments. >>>>> 4. Create the EntityManager object only once. >>>>> 5. Change new_element variable name >>>>> 6. Handle errors when committing the transaction. (RollBack when >>>>> an exception occurred.) >>>>> 7. Do a performance comparison between the current Smooks mediator >>>>> and improved mediator. >>>>> >>>>> >>>>> regards, >>>>> Lakmali >>>>> >>>> >>>> >>>> -- >>>> Lakmali Baminiwatta* >>>> * >>>> Software Engineer >>>> WSO2, Inc.: http://wso2.com >>>> lean.enterprise.middleware >>>> >>>> * >>>> * >>>> >>>> >>> >>> >>> -- >>> *Anjana Fernando* >>> Senior Software Engineer >>> WSO2 Inc. | http://wso2.com >>> lean . enterprise . middleware >>> >> >> >> >> -- >> Lakmali Baminiwatta* >> * >> Software Engineer >> WSO2, Inc.: http://wso2.com >> lean.enterprise.middleware >> >> * >> * >> >> > > > -- > *Anjana Fernando* > Senior Software Engineer > WSO2 Inc. | http://wso2.com > lean . enterprise . middleware > -- Lakmali Baminiwatta* * Software Engineer WSO2, Inc.: http://wso2.com lean.enterprise.middleware mobile: +94 71 2335936 * *
_______________________________________________ Dev mailing list [email protected] http://wso2.org/cgi-bin/mailman/listinfo/dev
