Skip to content

[ANNO-106]: Worker filter profiling #1081

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
Open
dchhabda wants to merge 4 commits into
base: main
Choose a base branch
from
Open

[ANNO-106]: Worker filter profiling #1081

dchhabda wants to merge 4 commits into from
+288 −8

Conversation

@dchhabda
Copy link

@dchhabda dchhabda commented Dec 29, 2025 •
edited
Loading

Changes covers

  • Logging around task assignments via task queue / user pref schedulers considering user profile and worker filter
  • Added tests that perform verification of task assignments considering user profile and tasks worker_filter
  • Potential changes around get_user_pref_task function that guarantees tasks always follows assignment considering filter_user_pref

@dchhabda dchhabda changed the title Test tok [ANNO-106]: Worker filter profiling Dec 29, 2025
@coveralls
Copy link

coveralls commented Dec 29, 2025 •
edited
Loading

Pull Request Test Coverage Report for Build 20608554074

Details

  • 21 of 21 (100.0%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.006%) to 94.073%
Totals Coverage Status
Change from base Build 20578716175: 0.006%
Covered Lines: 17902
Relevant Lines: 19030
💛 - Coveralls

@dchhabda dchhabda requested review from n00rsy and peterkle December 31, 2025 00:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@peterkle peterkle Awaiting requested review from peterkle

@n00rsy n00rsy Awaiting requested review from n00rsy

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dchhabda @coveralls