Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Tools] Update fix_candidate_age.php speed #9491

Merged
merged 1 commit into from
Dec 10, 2024
Merged

[Tools] Update fix_candidate_age.php speed #9491

merged 1 commit into from
Dec 10, 2024

Conversation

skarya22
Copy link
Contributor

Brief summary of changes

  • Updated the fix_candidate_age php tool to only consider candidates with Data, and to skip bulk loading when there are no participants

Testing instructions (if applicable)

  1. Try running the fix_candidate_age script and confirm it works

CCNA OVERRIDE

@maximemulder maximemulder added Project: CCNA Issue or PR related to the CCNA project Category: Perfomance Issue or PR that aims to report or improve perfomance and removed Priority: Projects labels Nov 29, 2024
@driusan driusan merged commit 2e4211f into aces:main Dec 10, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ridz1208 ridz1208 ridz1208 left review comments

@driusan driusan driusan left review comments

Assignees

@driusan driusan

Labels
Category: Perfomance Issue or PR that aims to report or improve perfomance Project: CCNA Issue or PR related to the CCNA project
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@skarya22 @driusan @ridz1208 @maximemulder