[CXF-8537] Replace String.replaceAll with substring for better throughput performance

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.3.10
• Fix Version/s: 3.5.0, 3.4.4
• Component/s: JAX-RS

Environment

This would affect all platforms.

• Estimated Complexity: Unknown
• Assignee: Freeman Yue Fang
• Reporter: Joshua Dettinger
• Created: 05/May/21 18:05
• Updated: 22/Dec/21 20:41
• Resolved: 06/May/21 19:50

Description

Sometime in the JAX-RS 3.3.x branch, the method org.apache.cxf.jaxrs.utils.JAXRSUtils.combineUriTemplates was introduced. This method calls a costly String.replaceAll in order to remove a '/' character at the end of a uri. This method can perform up to 2% better in throughput scenarios by changing:

return parent.replaceAll("/$", "") + child;

to

return parent.substring(0, parent.length() - 1) + child;