The StringUtils patch has resulted in an increase. Now it is 1.7% On Tue, Sep 4, 2012 at 8:15 PM, Afkham Azeez <az...@wso2.com> wrote:
> Your previous fix works. Now the method takes only 0.6% of the total > execution time. > > Azeez > > > On Tue, Sep 4, 2012 at 8:00 PM, Amani Soysa <am...@wso2.com> wrote: > >> I have found a better way of improving the performance of String replace >> by using org.apache.commons.lang.StringUtils.replace(serviceName, "/", >> "");. And attached the jar file and the diff to the jira. For more >> information on String replace performance [1] >> >> [1] - >> http://myhowto.org/under-the-hood/32-hidden-costs-of-using-java-string-methods/ >> >> >> On Tue, Sep 4, 2012 at 7:35 PM, Amani Soysa <am...@wso2.com> wrote: >> >>> >>> >>> On Tue, Sep 4, 2012 at 7:18 PM, Afkham Azeez <az...@wso2.com> wrote: >>> >>>> Anyway, how will the following improve performance? >>>> >>>> Java String.replace uses regular expressions and can be slow, >>> especially for large strings requiring many replacements. Therefore, i >>> thought of using substring instead of replace since according to the >>> diagram you've attached overhead of string substring is 0.1% for the same >>> string. I didn't test this properly however, I will update this thread as >>> soon as I am done. >>> >>> - serviceName = serviceName.replace("/", ""); >>>> + serviceName = serviceName.substring(serviceName.indexOf('/')+1, >>>> serviceName.length()); >>>> >>>> On Tue, Sep 4, 2012 at 7:15 PM, Afkham Azeez <az...@wso2.com> wrote: >>>> >>>>> I built a tomcat ext jar with your fix & replaced the original, and >>>>> still server hangs at startup. >>>>> >>>>> Azeez >>>>> >>>>> >>>>> On Tue, Sep 4, 2012 at 7:01 PM, Amani Soysa <am...@wso2.com> wrote: >>>>> >>>>>> Yes there's something wrong with the packs coming from the builder >>>>>> ... you need to have your own build .. My fix was a simple fix where i >>>>>> remove the String replace, that should not have any impact on server >>>>>> startup. >>>>>> >>>>>> On Tue, Sep 4, 2012 at 6:44 PM, Amila Maha Arachchi >>>>>> <ami...@wso2.com>wrote: >>>>>> >>>>>>> Did you take a pack from builder and replaced a jar with a one built >>>>>>> in your machine? >>>>>>> >>>>>>> Has it got something to do with the patch or does the bundle does >>>>>>> not get activated? Some of us have faced a problem which the packs built >>>>>>> from the builder doesn't get started when we replace the jars. I have >>>>>>> sent >>>>>>> a separate mail regarding this. >>>>>>> >>>>>>> AmilaM. >>>>>>> >>>>>>> On Tue, Sep 4, 2012 at 6:20 PM, Afkham Azeez <az...@wso2.com> wrote: >>>>>>> >>>>>>>> When I applied your patch, and tried to start the server, the >>>>>>>> server does not start. >>>>>>>> >>>>>>>> >>>>>>>> On Tue, Sep 4, 2012 at 3:19 PM, Amani Soysa <am...@wso2.com> wrote: >>>>>>>> >>>>>>>>> Fixed this issue [1]. I have attached the patch to the following >>>>>>>>> jira[1] >>>>>>>>> >>>>>>>>> [1] - https://wso2.org/jira/browse/CARBON-13859 >>>>>>>>> >>>>>>>>> >>>>>>>>> On Tue, Sep 4, 2012 at 2:46 PM, Amani Soysa <am...@wso2.com>wrote: >>>>>>>>> >>>>>>>>>> On Tue, Sep 4, 2012 at 2:41 PM, Afkham Azeez <az...@wso2.com>wrote: >>>>>>>>>> >>>>>>>>>>> Amani, Reka, >>>>>>>>>>> Please review the attached screenshot & fix. >>>>>>>>>>> >>>>>>>>>>> Will fix it. >>>>>>>>>> >>>>>>>>>>> Thanks >>>>>>>>>>> Azeez >>>>>>>>>>> >>>>>>>>>>> -- >>>>>>>>>>> *Afkham Azeez* >>>>>>>>>>> Director of Architecture; WSO2, Inc.; http://wso2.com >>>>>>>>>>> Member; Apache Software Foundation; http://www.apache.org/ >>>>>>>>>>> * <http://www.apache.org/>** >>>>>>>>>>> email: **az...@wso2.com* <az...@wso2.com>* cell: +94 77 3320919 >>>>>>>>>>> blog: **http://blog.afkham.org* <http://blog.afkham.org>* >>>>>>>>>>> twitter: >>>>>>>>>>> **http://twitter.com/afkham_azeez*<http://twitter.com/afkham_azeez> >>>>>>>>>>> * >>>>>>>>>>> linked-in: **http://lk.linkedin.com/in/afkhamazeez* >>>>>>>>>>> * >>>>>>>>>>> * >>>>>>>>>>> *Lean . Enterprise . Middleware* >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> -- >>>>>>>> *Afkham Azeez* >>>>>>>> Director of Architecture; WSO2, Inc.; http://wso2.com >>>>>>>> Member; Apache Software Foundation; http://www.apache.org/ >>>>>>>> * <http://www.apache.org/>** >>>>>>>> email: **az...@wso2.com* <az...@wso2.com>* cell: +94 77 3320919 >>>>>>>> blog: **http://blog.afkham.org* <http://blog.afkham.org>* >>>>>>>> twitter: >>>>>>>> **http://twitter.com/afkham_azeez*<http://twitter.com/afkham_azeez> >>>>>>>> * >>>>>>>> linked-in: **http://lk.linkedin.com/in/afkhamazeez* >>>>>>>> * >>>>>>>> * >>>>>>>> *Lean . Enterprise . Middleware* >>>>>>>> >>>>>>>> >>>>>>>> _______________________________________________ >>>>>>>> Dev mailing list >>>>>>>> Dev@wso2.org >>>>>>>> http://wso2.org/cgi-bin/mailman/listinfo/dev >>>>>>>> >>>>>>>> >>>>>>> >>>>>>> >>>>>>> -- >>>>>>> *Amila Maharachchi* >>>>>>> Technical Lead >>>>>>> Member, Management Committee - Cloud & Platform TG >>>>>>> WSO2, Inc.; http://wso2.com >>>>>>> >>>>>>> Blog: http://maharachchi.blogspot.com >>>>>>> Mobile: +94719371446 >>>>>>> >>>>>>> >>>>>>> >>>>>> >>>>> >>>>> >>>>> -- >>>>> *Afkham Azeez* >>>>> Director of Architecture; WSO2, Inc.; http://wso2.com >>>>> Member; Apache Software Foundation; http://www.apache.org/ >>>>> * <http://www.apache.org/>** >>>>> email: **az...@wso2.com* <az...@wso2.com>* cell: +94 77 3320919 >>>>> blog: **http://blog.afkham.org* <http://blog.afkham.org>* >>>>> twitter: >>>>> **http://twitter.com/afkham_azeez*<http://twitter.com/afkham_azeez> >>>>> * >>>>> linked-in: **http://lk.linkedin.com/in/afkhamazeez* >>>>> * >>>>> * >>>>> *Lean . Enterprise . Middleware* >>>>> >>>>> >>>> >>>> >>>> -- >>>> *Afkham Azeez* >>>> Director of Architecture; WSO2, Inc.; http://wso2.com >>>> Member; Apache Software Foundation; http://www.apache.org/ >>>> * <http://www.apache.org/>** >>>> email: **az...@wso2.com* <az...@wso2.com>* cell: +94 77 3320919 >>>> blog: **http://blog.afkham.org* <http://blog.afkham.org>* >>>> twitter: >>>> **http://twitter.com/afkham_azeez*<http://twitter.com/afkham_azeez> >>>> * >>>> linked-in: **http://lk.linkedin.com/in/afkhamazeez* >>>> * >>>> * >>>> *Lean . Enterprise . Middleware* >>>> >>>> >>> >> > > > -- > *Afkham Azeez* > Director of Architecture; WSO2, Inc.; http://wso2.com > Member; Apache Software Foundation; http://www.apache.org/ > * <http://www.apache.org/>** > email: **az...@wso2.com* <az...@wso2.com>* cell: +94 77 3320919 > blog: **http://blog.afkham.org* <http://blog.afkham.org>* > twitter: **http://twitter.com/afkham_azeez*<http://twitter.com/afkham_azeez> > * > linked-in: **http://lk.linkedin.com/in/afkhamazeez* > * > * > *Lean . Enterprise . Middleware* > > -- *Afkham Azeez* Director of Architecture; WSO2, Inc.; http://wso2.com Member; Apache Software Foundation; http://www.apache.org/ * <http://www.apache.org/>** email: **az...@wso2.com* <az...@wso2.com>* cell: +94 77 3320919 blog: **http://blog.afkham.org* <http://blog.afkham.org>* twitter: **http://twitter.com/afkham_azeez*<http://twitter.com/afkham_azeez> * linked-in: **http://lk.linkedin.com/in/afkhamazeez* * * *Lean . Enterprise . Middleware*
_______________________________________________ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev