Comment 2 for bug 1397445

Revision history for this message
yunfei (233602551-t) wrote :

Hi, chjgcn. I read some code of your patch, I found a little problem.

In libgearman-server/job.cc , The function : gearman_server_job_queue.

" while (worker != job->function->worker_list && (worker_wakeup == 0 || worker_wakeup < noop_sent));" This line.

Are you sure the condition " worker_wakeup < noop_sent" is right?

Maybe it's "noop_sent < worker_wakeup ". Please confirm this, thx!