Improve performance for large pipeline graphs

johannesacco commented 2 months ago

Make sure that logs are activated before calling expensive function calls. Also use StringBuilder to format Strings. This will improve performance drastically (~10 s -> ~300 ms in one test scenario).

Testing done

Executed on a local installation.

Fixes #385

Submitter checklist

[X] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[X] Ensure that the pull request title represents the desired changelog entry
[X] Please describe what you did
[X] Link to relevant issues in GitHub or Jira
[ ] Link to relevant pull requests, esp. upstream and downstream changes
[ ] Ensure you have provided tests - that demonstrates feature works or fixes the issue

smaduel-dn commented 2 months ago

@timja - When will we have released version with this fix?

timja commented 2 months ago

https://github.com/jenkinsci/pipeline-graph-view-plugin/releases/tag/340.v28cecee8b_25f

johannesacco commented 2 months ago

Wow, that was fast, thanks!

jenkinsci / pipeline-graph-view-plugin

Improve performance for large pipeline graphs #503

Testing done

Submitter checklist