X-Git-Url: https://git.novaco.in/?p=electrum-server.git;a=blobdiff_plain;f=processor.py;h=b94231b85e587f74d818f58e90fe07eebef4860b;hp=ec62d482686544e77a00d196048fb9c174f20125;hb=240db7c28d2e6a0d0078c42f81b2b110615e88c6;hpb=aa8454ee6f18f592e3d679cfde036fd77da111ab diff --git a/processor.py b/processor.py index ec62d48..b94231b 100644 --- a/processor.py +++ b/processor.py @@ -34,21 +34,21 @@ class Processor(threading.Thread): self.dispatcher = None self.queue = queue.Queue() - def process(self, request): + def process(self, session, request): pass - def add_request(self, request): - self.queue.put(request) + def add_request(self, session, request): + self.queue.put((session, request)) - def push_response(self, response): + def push_response(self, session, response): #print "response", response - self.dispatcher.request_dispatcher.push_response(response) + self.dispatcher.request_dispatcher.push_response(session, response) def run(self): while not self.shared.stopped(): - request = self.queue.get(10000000000) + request, session = self.queue.get(10000000000) try: - self.process(request) + self.process(request, session) except: traceback.print_exc(file=sys.stdout) @@ -80,15 +80,13 @@ class RequestDispatcher(threading.Thread): self.daemon = True self.request_queue = queue.Queue() self.response_queue = queue.Queue() - self.internal_ids = {} - self.internal_id = 1 self.lock = threading.Lock() self.idlock = threading.Lock() - self.sessions = [] + self.sessions = {} self.processors = {} - def push_response(self, item): - self.response_queue.put(item) + def push_response(self, session, item): + self.response_queue.put((session, item)) def pop_response(self): return self.response_queue.get() @@ -100,24 +98,16 @@ class RequestDispatcher(threading.Thread): return self.request_queue.get() def get_session_by_address(self, address): - for x in self.sessions: + for x in self.sessions.values(): if x.address == address: return x - def get_session_id(self, internal_id): - with self.idlock: - return self.internal_ids.pop(internal_id) - - def store_session_id(self, session, msgid): - with self.idlock: - self.internal_ids[self.internal_id] = session, msgid - r = self.internal_id - self.internal_id += 1 - return r - def run(self): if self.shared is None: raise TypeError("self.shared not set in Processor") + + lastgc = 0 + while not self.shared.stopped(): session, request = self.pop_request() try: @@ -125,6 +115,10 @@ class RequestDispatcher(threading.Thread): except: traceback.print_exc(file=sys.stdout) + if time.time() - lastgc > 60.0: + self.collect_garbage() + lastgc = time.time() + self.stop() def stop(self): @@ -141,9 +135,6 @@ class RequestDispatcher(threading.Thread): if suffix == 'subscribe': session.subscribe_to_service(method, params) - # store session and id locally - request['id'] = self.store_session_id(session, request['id']) - prefix = request['method'].split('.')[0] try: p = self.processors[prefix] @@ -151,7 +142,7 @@ class RequestDispatcher(threading.Thread): print_log("error: no processor for", prefix) return - p.add_request(request) + p.add_request(session, request) if method in ['server.version']: session.version = params[0] @@ -163,35 +154,32 @@ class RequestDispatcher(threading.Thread): def get_sessions(self): with self.lock: - r = self.sessions[:] + r = self.sessions.values() return r def add_session(self, session): + key = session.key() with self.lock: - self.sessions.append(session) + self.sessions[key] = session - def collect_garbage(self): - # Deep copy entire sessions list and blank it - # This is done to minimize lock contention + def remove_session(self, session): + key = session.key() with self.lock: - sessions = self.sessions[:] + self.sessions.pop(key) - active_sessions = [] + def collect_garbage(self): now = time.time() - for session in sessions: - if not session.stopped() and (now - session.time) < 1000: - # If session is still alive then re-add it back - # to our internal register - active_sessions.append(session) - - with self.lock: - self.sessions = active_sessions[:] + for session in self.sessions.values(): + if (now - session.time) > session.timeout: + session.stop() class Session: - def __init__(self): + def __init__(self, dispatcher): + self.dispatcher = dispatcher + self.bp = self.dispatcher.processors['blockchain'] self._stopped = False self.lock = threading.Lock() self.subscriptions = [] @@ -202,6 +190,11 @@ class Session: self.time = time.time() threading.Timer(2, self.info).start() + + def key(self): + return self.name + self.address + + # Debugging method. Doesn't need to be threadsafe. def info(self): for sub in self.subscriptions: @@ -220,34 +213,42 @@ class Session: "%3d" % len(self.subscriptions), self.version) + def stop(self): + with self.lock: + if self._stopped: + return + self._stopped = True + + self.shutdown() + self.dispatcher.remove_session(self) + self.stop_subscriptions() + + + def shutdown(self): + pass + + def stopped(self): with self.lock: return self._stopped + def subscribe_to_service(self, method, params): - subdesc = self.build_subdesc(method, params) with self.lock: - if subdesc is not None: - self.subscriptions.append(subdesc) - - # subdesc = A subscription description - @staticmethod - def build_subdesc(method, params): - if method == "blockchain.numblocks.subscribe": - return method, - elif method == "blockchain.headers.subscribe": - return method, - elif method in ["blockchain.address.subscribe"]: - if not params: - return None - else: - return method, params[0] - else: - return None + if self._stopped: + return + if (method, params) not in self.subscriptions: + self.subscriptions.append((method,params)) + self.bp.do_subscribe(method, params, self) - def contains_subscription(self, subdesc): + + def stop_subscriptions(self): + with self.lock: + s = self.subscriptions[:] + for method, params in s: + self.bp.do_unsubscribe(method, params, self) with self.lock: - return subdesc in self.subscriptions + self.subscriptions = [] class ResponseDispatcher(threading.Thread): @@ -260,48 +261,5 @@ class ResponseDispatcher(threading.Thread): def run(self): while not self.shared.stopped(): - self.update() - - def update(self): - response = self.request_dispatcher.pop_response() - #print "pop response", response - internal_id = response.get('id') - method = response.get('method') - params = response.get('params') - - # A notification - if internal_id is None: # and method is not None and params is not None: - found = self.notification(method, params, response) - if not found and method == 'blockchain.address.subscribe': - request = { - 'id': None, - 'method': method.replace('.subscribe', '.unsubscribe'), - 'params': [self.shared.config.get('server', 'password')] + params, - } - - self.request_dispatcher.push_request(None, request) - # A response - elif internal_id is not None: - self.send_response(internal_id, response) - else: - print_log("no method", response) - - def notification(self, method, params, response): - subdesc = Session.build_subdesc(method, params) - found = False - for session in self.request_dispatcher.sessions: - if session.stopped(): - continue - if session.contains_subscription(subdesc): - session.send_response(response) - found = True - # if not found: print_log("no subscriber for", subdesc) - return found - - def send_response(self, internal_id, response): - session, message_id = self.request_dispatcher.get_session_id(internal_id) - if session: - response['id'] = message_id + session, response = self.request_dispatcher.pop_response() session.send_response(response) - #else: - # print_log("send_response: no session", message_id, internal_id, response )