SONARJAVA-7182 Replace frequently allocated streams with loops

[aurelien-coet-sonarsource]

---

Summary by Gitar

• Refactoring:
  • Replaced stream-based anyMatch calls in MethodMatchersList with a loop-based matchesAny helper
  • Replaced the Stream.of pipeline in JMethodSymbol.findOverridesInTypes with a for loop that breaks on first match

_{This will update automatically on new commits.}

[hashicorp-vault-sonar-prod]

SONARJAVA-7182

[aurelien-coet-sonarsource]

Bechmark:

To assess the impact of the changes in this PR, a benchmark was run on a sample project (the core module in jOOQ, 2,217 Java files) before and after the changes.

The benchmark repeatedly analyzed the same 2,127 jOOQ Java files with unchanged rules and settings. Each version had ten measured scans at both 2 GB and 4 GB heaps, using fresh JVMs, plus three separate 4 GB Java Flight Recorder allocation profiles. JFR estimated allocated bytes by object class and call stack; GC logs measured collection activity and heap occupancy, while process sampling measured CPU and peak RSS. Results below are medians, and allocation volume represents cumulative traffic rather than retained memory.

Results:

Replacing Streams with loops in JMethodSymbol.findOverridesInTypes() and MethodMatchersList.matches() substantially reduced allocation traffic:

Median per scan	Original Streams	Loops	Change
Total allocation	91.31 GB	82.39 GB	−9.8%
Stream-object allocation	16.10 GB	7.89 GB	−51.0%
Java analysis, 2 GB heap	103.9 s	101.1 s	−2.7%
Java analysis, 4 GB heap	94.1 s	93.2 s	−1.0%

Both modified methods had zero sampled Stream allocations after replacement. GC pause counts fell by approximately 8%, but total pause time and peak RSS changed little.

The clearest improvement was lower allocation volume, rather than lower peak memory or a reliably established speedup. Both versions produced identical analysis results.

[gitar-bot]

Code Review ✅ Approved

🟢 Low risk · Replacing two stream pipelines with equivalent loops is narrowly scoped and reduces allocation without changing matching or override-selection behavior.

Replaces stream-based anyMatch calls in MethodMatchersList with a loop-based matchesAny helper, and swaps the Stream.of pipeline in JMethodSymbol.findOverridesInTypes for a for loop that breaks on first match. No issues found.

Verification Rules 2/2 met

Functional validation No results

Rules 2 passed 1 not applicable

✅ Internal artifact registries
✅ Repository license compliance

Auto-approval Not enabled

Set up

_{For more verification details, comment gitar display:verbose.}

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing compact verification results.

Comment with these commands to change the behavior for this request:

Auto-apply	Compact
gitar auto-apply:on	gitar display:verbose

_{Was this helpful? React with 👍 / 👎 | Gitar}

[sonarqube-next]

Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0 Dependency risks
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube

[vdiez]

LGTM