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

Add support for DATA packets #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

issacg
Copy link

@issacg issacg commented Oct 14, 2012

This patch should allow very rudementary support for WORK_DATA packets. At the very least it won't cause Gearman::Client to die with "Unknown/unimplemented packet type" when one is encountered.

@dwbuiten
Copy link

Ping. This is quite an important fix.

Is this implementation unmaintained?

@p-alik
Copy link

p-alik commented Oct 26, 2016

@issacg, I'm a new Gearman::Client co maintainer. I'm working on the project in my own repo because lack of permissions for this repo. It's a pity that I didn't realize your PL sooner. It would be great if you could create a new PL on my repo. Otherwise I'll do it on my own.

@issacg
Copy link
Author

issacg commented Oct 27, 2016

@p-alik I don't have cycles free at the moment to port it - if you have the time and availability to do it, you have my blessings :) Thanks!

p-alik added a commit to p-alik/perl-Gearman that referenced this pull request Oct 27, 2016
p-alik added a commit to p-alik/perl-Gearman that referenced this pull request Oct 27, 2016
@p-alik
Copy link

p-alik commented Oct 27, 2016

thanks @issacg I did it

p-alik added a commit to p-alik/perl-Gearman that referenced this pull request Jan 28, 2017
p-alik added a commit to p-alik/perl-Gearman that referenced this pull request Jan 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants