Skip to content

Fix performance of task "releases" #4034

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 10, 2025
Merged

Conversation

kszymukowicz
Copy link
Contributor

@kszymukowicz kszymukowicz commented Mar 9, 2025

Fix performance of task releases by checking cat releases/$release/REVISION only for existing releases and not all past releases stored in releases_log.

A real example data to compare results.
In releases_log there are 160 releases logs.
In releases_list there are 5 releases folders.

Before fix: 57 seconds.
After fix: 7 seconds.

  • Bug fix
  • New feature?
  • BC breaks?
  • Tests added?
  • Docs added?

…/REVISION" only for existing releases and not all past releases stored in "releases_log".
@antonmedv
Copy link
Member

Nice!

@antonmedv antonmedv merged commit 218b291 into deployphp:master Mar 10, 2025
6 checks passed
@duhui
Copy link

duhui commented Mar 12, 2025

Hi @antonmedv will this be merged into 7.x branch?

@antonmedv
Copy link
Member

Nope =/

kszymukowicz added a commit to sourcebroker/deployer-extended that referenced this pull request Mar 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants