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

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

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

Nicolas Malin (Jira)
Jacques Le Roux created OFBIZ-7926:
--------------------------------------

             Summary: 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
             Fix For: Upcoming Branch


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
(v6.3.4#6332)