Skip to content

Commit 5ef18fd

Browse files
committed
Merge pull request #737 from hulu/master
add daemon option to PubSubWorker so ctrl-c will work
2 parents 273a47e + 226b290 commit 5ef18fd

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

redis/client.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2362,22 +2362,23 @@ def handle_message(self, response, ignore_subscribe_messages=False):
23622362

23632363
return message
23642364

2365-
def run_in_thread(self, sleep_time=0):
2365+
def run_in_thread(self, sleep_time=0, daemon=False):
23662366
for channel, handler in iteritems(self.channels):
23672367
if handler is None:
23682368
raise PubSubError("Channel: '%s' has no handler registered")
23692369
for pattern, handler in iteritems(self.patterns):
23702370
if handler is None:
23712371
raise PubSubError("Pattern: '%s' has no handler registered")
23722372

2373-
thread = PubSubWorkerThread(self, sleep_time)
2373+
thread = PubSubWorkerThread(self, sleep_time, daemon=daemon)
23742374
thread.start()
23752375
return thread
23762376

23772377

23782378
class PubSubWorkerThread(threading.Thread):
2379-
def __init__(self, pubsub, sleep_time):
2379+
def __init__(self, pubsub, sleep_time, daemon=False):
23802380
super(PubSubWorkerThread, self).__init__()
2381+
self.daemon = daemon
23812382
self.pubsub = pubsub
23822383
self.sleep_time = sleep_time
23832384
self._running = False

0 commit comments

Comments
 (0)