[SPARK-48496] Use static regex Pattern instances in common/utils JavaUtils

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 4.0.0
• Fix Version/s: 4.0.0
• Component/s: Spark Core
• Labels: pull-request-available
• Assignee: Josh Rosen
• Reporter: Josh Rosen
• Created: 31/May/24 21:53
• Updated: 03/Jun/24 01:58
• Resolved: 03/Jun/24 01:58

Description

Some methods in JavaUtils.java are recompiling regexes on every invocation; we should instead store a single cached Pattern.

This is a minor perf. issue that I spotted in the context of other profiling. Not a huge bottleneck in the grand scheme of things, but simple and straightforward to fix.

Issue Links

• links to: GitHub Pull Request #46829