perf: size the workflow result map from Workflow#size

[csviri]

AbstractWorkflowExecutor called Workflow#getDependentResourcesByName purely to
read its size. That allocates a HashMap and walks every node to collect the
dependent resources, then discards the map. The executor is constructed on
every reconcile and cleanup of a workflow-based reconciler, so use the existing
Workflow#size instead.

---

Quality-only change: no intended behavior difference. Cut from next and
touches a disjoint set of files from the sibling cleanup PRs, so it can be merged
independently and in any order.

Verified on this branch alone: mvn -o -pl operator-framework-core,operator-framework-junit -am test
(693 core + 6 junit tests, no failures) and mvn spotless:check.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 8c0cc15a-cbdb-4986-8828-c6b1c4e28485

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[Copilot]

Pull request overview

This PR reduces per-reconcile allocation overhead in the workflow execution path by avoiding creation of a dependent-resources map solely to pre-size the executor’s result map. It leverages DefaultWorkflow#size() (which is backed by dependentResourceNodes.size()) to keep behavior unchanged while eliminating unnecessary work.

Changes:

• Replace workflow.getDependentResourcesByName().size() with workflow.size() when initializing results in AbstractWorkflowExecutor.
• Avoid allocating and populating a temporary HashMap of dependent resources during executor construction.

[csviri]

LGTM