[jira] [Updated] (OFBIZ-7926) Replace Jakarta ORO by java.util.regex

["Deepak Dixit (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/OFBIZ-7926?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Deepak Dixit updated OFBIZ-7926:
--------------------------------
    Fix Version/s:     (was: 17.12.01)

> Replace Jakarta ORO by  java.util.regex
> ---------------------------------------
>
>                 Key: OFBIZ-7926
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-7926
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework, product
>    Affects Versions: Trunk
>            Reporter: Jacques Le Roux
>            Assignee: Jacques Le Roux
>            Priority: Major
>
> At OFBIZ-5395 I discussed with Adrian the possibility to replace Jakarta ORO by  java.util.regex. At this time, based on arguments given by http://tusker.org/regex/regex_benchmark.html in a bottom comment I stated
> bq. ORO is 6 times faster than Java 1.6 regex (see link above). That's why I want to change the Java regex usage to ORO in OFBiz when relevant and possible (ORO has not all the features of Java regex). No needs to worry about it, it's fast and reliable. Anyway, If ever in the future we want to move to Java regex it's not a big deal. I saw you commited PatternFactory.java, I will have a look at the rest when I will get a chance.
> More recently by chance I stumbled upon http://www.javaadvent.com/2015/12/java-regular-expression-library-benchmarks-2015.html and in a last comment at OFBIZ-5395 I wrote (shortened)
> bq. Just an update about "Java Regular Expression Library Benchmarks" in 2015, Tusker was in 2010!
> So when you read the resuilt of the "Java Regular Expression Library Benchmarks" you can see it's time to switch from ORO. I'll have though to carefully revisit the end of OFBIZ-5395. IIRW no concurrency issue should be expected, they were removed by concurrency OFBIZ-5395.



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