Jesse Glick created MNG-5314:
--------------------------------

             Summary: DefaultModelValidator misuses String.matches
                 Key: MNG-5314
                 URL: https://jira.codehaus.org/browse/MNG-5314
             Project: Maven 2 & 3
          Issue Type: Bug
          Components: Bootstrap & Build
    Affects Versions: 3.0.4
            Reporter: Jesse Glick
            Priority: Minor
         Attachments: DefaultModelValidator.diff

While investigating MNG-5312, a profiler run turned up tens of thousands of 
calls to {{DefaultModelValidator.validateId}} consuming a considerable portion 
of total CPU time. Turns out the existing code uses {{String.matches}}, which 
recompiles the regex on each call, which is far more expensive than applying a 
precompiled pattern. A trivial fix makes this method disappear from the hot 
spots shown by the profiler, though the fix of MNG-5312 of course means that 
there would be far fewer calls to this method to begin with.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://jira.codehaus.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to