Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • B bull
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 175
    • Issues 175
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 9
    • Merge requests 9
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • OptimalBits
  • bull
  • Issues
  • #651
Closed
Open
Issue created Aug 12, 2017 by Manuel Astudillo@manastContributor

[enhancement] use BRPOPLPUSH instead of BLPOP for notifying waiting jobs

Currently we have a special list wait:added used for notifying that new jobs have arrived to the queue. This works well but can be done better. If we use BRPOPLPUSH instead but using the same source and destination, we can have only one list and get rid of wait:added, this simplifies code but also has another advantage, we can call moveToActive directly to avoid an extra call to BLPOP, for instance if the queue has not drained we can just keep moving jobs to the active list faster.

Assignee
Assign to
Time tracking