[
https://issues.apache.org/jira/browse/VALIDATOR-252?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Ben Speakmon resolved VALIDATOR-252.
------------------------------------
Resolution: Fixed
Assignee: Ben Speakmon
Patch applied. Thanks!
$ svn ci -m "VALIDATOR-252: improve regex performance for DomainValidator"
Sending
src/main/java/org/apache/commons/validator/routines/DomainValidator.java
Transmitting file data .
Committed revision 600231.
> Performance improvement of DomainValidator by change the regular expression
> ---------------------------------------------------------------------------
>
> Key: VALIDATOR-252
> URL: https://issues.apache.org/jira/browse/VALIDATOR-252
> Project: Commons Validator
> Issue Type: Improvement
> Components: Routines
> Affects Versions: 1.4
> Environment: Windows 2000 Professional, Pentium4 2.4GHz, RAM:512M,
> Eclipse 3.3m Junit 3.8.1
> Reporter: Makoto Uchino
> Assignee: Ben Speakmon
> Priority: Minor
> Fix For: 1.4
>
> Attachments: DomainValidator.java.diff
>
>
> It seemed to take very long time to match the regular expression when long
> TLD (consists of around 30-characters over TLD) was specified for
> DomainValidator, this exponential performance will make it appear that the
> program has hung.
> The match time improves if it is possible to change DOMAIN_LABEL_REGEX from
> using (?:) to (?>), as follows.
> DOMAIN_LABEL_REGEX = "\\p{Alnum}(?>[\\p{Alnum}-]*\\p{Alnum})*"
> (All unit tests in DomainValidatorTest are succeed even if it changes as
> shown in the above-mentioned.)
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
