Simple question about Queue.Queue and threads

F

Frank Millman

Hi all

Assume you have a server process running, a pool of worker threads to
perform tasks, and a Queue.Queue() to pass the tasks to the workers.

In order to shut down the server cleanly, you want to ensure that the
workers have all finished their tasks. I like the technique of putting a
None onto the queue, and have each worker check for None, put None back onto
the queue, and terminate itself.

The main program would look something like this -

q.put(None)
for worker in worker_threads:
worker.join()

At this point you can be sure that each thread has completed its tasks and
terminated itself.

However, the queue is not empty - it still has the final None in it.

Is it advisable to finalise the cleanup like this? -

while not q.empty():
q.get()
q.task_done()
q.join()

Or is this completely redundant?

Thanks

Frank Millman
 
S

Steven

Hi all

Assume you have a server process running, a pool of worker threads to
perform tasks, and aQueue.Queue() to pass the tasks to the workers.

In order to shut down the server cleanly, you want to ensure that the
workers have all finished their tasks. I like the technique of putting a
None onto thequeue, and have each worker check for None, put None back onto
thequeue, and terminate itself.

The main program would look something like this -

    q.put(None)
    for worker in worker_threads:
        worker.join()

At this point you can be sure that each thread has completed its tasks and
terminated itself.

However, thequeueis not empty - it still has the final None in it.

Is it advisable to finalise the cleanup like this? -

    while not q.empty():
        q.get()
        q.task_done()
    q.join()

Or is this completely redundant?

Thanks

Frank Millman

Queue objects have support for this signaling baked in with
q.task_done and q.join.

After the server process has put all tasks into the queue, it can join
the queue itself, not the worker threads.

q.join()

This will block until all tasks have been gotten AND completed. The
worker threads would simply do this:
task_data = q.get()
do_task(task_data)
q.task_done()

Using pairs of get and task_done you no longer need to send a signal.
Just exit the server process and the worker threads will die (assuming
of course, you set .setDaemon(True) before starting each worker
thread).

Steven Rumbalski
 
D

Dennis Lee Bieber

Queue objects have support for this signaling baked in with
q.task_done and q.join.
Only in Python 2.5 and later (and I, for one, only upgraded to 2.5
last summer; suspect 2.6 will be chosen later this year). Passing a
unique sentinel works in all versions supporting Queue. And if a second
Queue is used for returns from the threads, each expiring thread could
return its ID for use in a .join() operation -- whereas using the
q.join() method blocks the collector of the return data until all the
tasks are done.
 
F

Frank Millman

Queue objects have support for this signaling baked in with
q.task_done and q.join.

After the server process has put all tasks into the queue, it can join
the queue itself, not the worker threads.

q.join()

This will block until all tasks have been gotten AND completed. The
worker threads would simply do this:
task_data = q.get()
do_task(task_data)
q.task_done()

Using pairs of get and task_done you no longer need to send a signal.
Just exit the server process and the worker threads will die (assuming
of course, you set .setDaemon(True) before starting each worker
thread).

Thanks, Steven.

This works perfectly in my scenario, and tidies up the code a bit.

Minor point - according to the 2.6 docs, .setDaemon(True) is the old API -
the current way of specifying this is .daemon = True.

Thanks for the tip.

Frank
 

Ask a Question

Want to reply to this thread or ask your own question?

You'll need to choose a username for the site, which only take a couple of moments. After that, you can post your question and our members will help you out.

Ask a Question

Members online

Forum statistics

Threads
473,955
Messages
2,570,125
Members
46,706
Latest member
liammiller

Latest Threads

Top