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.

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

Implement priorities for redis queues #61

Open
nickpeirson opened this issue Dec 7, 2016 · 1 comment
Open

Implement priorities for redis queues #61

nickpeirson opened this issue Dec 7, 2016 · 1 comment
Labels

Comments

@nickpeirson
Copy link

Currently the priority argument for a job is silently ignored for redis and jobs replayed from redis are all added at the normal priority.

A possible solution would be to use hashes. If instead of calling SET key data the code called HMSET key data "data" priority "priority", and HGETALL key in the replay method, it would be possible to store and retrieve the priority from redis.

@p-alik
Copy link
Collaborator

p-alik commented Dec 7, 2016

Indeed it's a bug in redis/queue.cc. gearman_job_priority_t parameter is undefined.

@p-alik p-alik added the bug label Dec 7, 2016
p-alik added a commit to p-alik/gearmand that referenced this issue Jan 9, 2017
- use redis HMSET/HGETALL commands to put/fetch  data and priority
@p-alik p-alik mentioned this issue Jan 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants