[CTAKES-531] String.replaceAll() has bad performance against replace() if not regex.

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: future enhancement
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: bd2019us
• Created: 14/Apr/19 00:23
• Updated: 20/Dec/22 20:39
• Resolved: 20/Dec/22 20:39

Description

Location: 

(1) ctakes-core/src/main/java/org/apache/ctakes/core/cc/html/HtmlTextWriter.java

(2) ctakes-gui/src/main/java/org/apache/ctakes/gui/dictionary/util/FileUtil.java

 

The String.replaceAll(regex, other) can replace the string satisfying the "regex" with the given "other" string. However, if the given "regex" is a simple plain string (i.e., no special characters), another API String.replace(origin, other) is recommended as the former need to be compiled a head of time, which will cause additional overhead.