can_do_timeout support missing.

Yelp / python-gearman

Gearman API - Client, worker, and admin client interfaces

http://github.com/Yelp/python-gearman/

Other

242 stars 122 forks source link

Closed JustinAzoff closed 14 years ago

JustinAzoff commented 14 years ago

the optional timeout option to register_task is missing... basically the worker can support

CAN_DO task

and

CAN_DO_TIMEOUT task timeout

If you don't have time to add this I can probably cook up a patch...

mtai commented 14 years ago

I can add this, but I don't think the C-gearman server actually does anything with it. Try grepping libgearman-server for:

worker->timeout

Is this something you'd still like? If so I can patch in your request

mtai commented 14 years ago

Ping! Still interested in this implementation?

JustinAzoff commented 14 years ago

Yeah.. though as you pointed out the C server doesn't support it either, which makes it kind of pointless..