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

libgearman isn't hashing submitted jobs based on unique #64

Open
SpamapS opened this issue Dec 7, 2016 · 5 comments
Open

libgearman isn't hashing submitted jobs based on unique #64

SpamapS opened this issue Dec 7, 2016 · 5 comments

Comments

@SpamapS
Copy link
Member

SpamapS commented Dec 7, 2016

In the original gearman client, the client would take the list of servers it knows about, hash the unique ID of each submitted job, and choose one of the servers in the list based on the hash of the unique.

This code seems to have been lost, and no tests enforce it. This is an important part of what makes gearman awesome, so it needs to be resurrected.

@SpamapS SpamapS added the bug label Dec 7, 2016
@sirgalleto
Copy link

@SpamapS I actually have a bug with the queue creation with mysql because the queue is trying to create a duplicate entry in the database. Please tell me if we are talking about the same bug and if you found any kind of solution.

Thanks

@p-alik
Copy link
Collaborator

p-alik commented Mar 1, 2017

@sirgalleto, see mein comment in the isseu #80

@SpamapS
Copy link
Member Author

SpamapS commented Mar 1, 2017

Indeed, @sirgalleto this is not related. This is about clients of many gearmand's that are selecting the wrong gearmand to submit their job to.

@p-alik
Copy link
Collaborator

p-alik commented May 8, 2017

In the original gearman client, the client would take the list of servers it knows about, hash the unique ID of each submitted job, and choose one of the servers in the list based on the hash of the unique.

@SpamapS, could you point me to the code part you mentioned here?

@bmeynell
Copy link

@p-alik - This issue appears to have been discovered here and first logged here. This is a huuuuuge gaping hole preventing gearmand from being a true "distributed nervous system". I would also be keen to see the portion(s) of the C code that would need modifications if anyone has that on their radar.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants