You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@ofbiz.apache.org by "bd2019us (JIRA)" <ji...@apache.org> on 2019/05/05 18:32:00 UTC

[jira] [Comment Edited] (OFBIZ-10934) Performance Increase: Using replace() instead of replaceAll() when a regex is not used increases performance

    [ https://issues.apache.org/jira/browse/OFBIZ-10934?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16833420#comment-16833420 ] 

bd2019us edited comment on OFBIZ-10934 at 5/5/19 6:31 PM:
----------------------------------------------------------

Apologies for the delayed reply. I intend to apply this change to some of the above 87 occurrences, in the upcoming days. The next update to this request from me will include those additional patches.


was (Author: bd2019us):
Apologies for the delayed information. I intend to apply this change to some of the above 87 occurrences, in the upcoming days. The next update to this request from me will include those additional patches.

> Performance Increase: Using replace() instead of replaceAll() when a regex is not used increases performance
> ------------------------------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-10934
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-10934
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: Trunk
>            Reporter: bd2019us
>            Priority: Trivial
>              Labels: pull-request-available
>         Attachments: OFBIZ-10934.patch
>
>
> Affected files:
> # framework/base/src/main/java/org/apache/ofbiz/base/util/UtilHttp.java
> # framework/widget/src/main/java/org/apache/ofbiz/widget/renderer/macro/MacroFormRenderer.java
> # framework/entity/src/main/java/org/apache/ofbiz/entity/condition/EntityOperator.java
> # framework/base/src/main/java/org/apache/ofbiz/base/util/KeyStoreUtil.java 
> When replaceAll() is utilized and no regex is used, replaceAll() can be replaced with replace() for better performance.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)