From be1dde95a65b6ace5d8a44eeffaa46fb8598fbcd Mon Sep 17 00:00:00 2001 From: Kevin F Date: Sun, 20 Mar 2022 12:54:57 -0500 Subject: [PATCH] Corrected type hinting for block queues and peer sets --- src/gossip/__init__.py | 6 +++--- src/gossip/client/__init__.py | 4 ++-- src/gossip/client/storeblocks.py | 6 +++--- src/gossip/server/__init__.py | 4 ++-- src/gossip/server/acceptstem.py | 2 +- 5 files changed, 11 insertions(+), 11 deletions(-) diff --git a/src/gossip/__init__.py b/src/gossip/__init__.py index 38207bf6..6ebcfad6 100644 --- a/src/gossip/__init__.py +++ b/src/gossip/__init__.py @@ -2,11 +2,11 @@ import threading from time import sleep from typing import TYPE_CHECKING, Set, Tuple from os import urandom -import queue if TYPE_CHECKING: from ordered_set import OrderedSet from onionrblocks import Block + from queue import Queue from .peer import Peer @@ -41,8 +41,8 @@ In stem phase, server disables diffusion def start_gossip_threads( - peer_set: OrderedSet['Peer'], - block_queues: Tuple[queue.Queue['Block']]): + peer_set: "OrderedSet[Peer]", + block_queues: Tuple["Queue[Block]"]): # Peer set is largely handled by the transport plugins # There is a unified set so gossip logic is not repeated seed = urandom(32) diff --git a/src/gossip/client/__init__.py b/src/gossip/client/__init__.py index fe887daa..d7b881ee 100644 --- a/src/gossip/client/__init__.py +++ b/src/gossip/client/__init__.py @@ -50,8 +50,8 @@ along with this program. If not, see . def gossip_client( - peer_set: OrderedSet['Peer'], - block_queues: Tuple[Queue['Block'], Queue['Block']], + peer_set: "OrderedSet[Peer]", + block_queues: Tuple["Queue[Block]", "Queue[Block]"], dandelion_seed: bytes): """ Gossip client does the following: diff --git a/src/gossip/client/storeblocks.py b/src/gossip/client/storeblocks.py index 44def179..1ef5471f 100644 --- a/src/gossip/client/storeblocks.py +++ b/src/gossip/client/storeblocks.py @@ -10,12 +10,12 @@ if TYPE_CHECKING: def store_blocks( - block_queues: Tuple[Queue['Block'], Queue['Block']], + block_queues: Tuple["Queue[Block]", "Queue[Block]"], dandelion_phase: 'DandelionPhase'): - new_queue: Queue['Block'] = Queue() + new_queue: "Queue[Block]" = Queue() - def _watch_queue(block_queue: Queue['Block']): + def _watch_queue(block_queue: "Queue[Block]"): # Copy all incoming blocks into 1 queue which gets processed to db while not dandelion_phase.is_stem_phase() \ and dandelion_phase.remaining_time() > 1: diff --git a/src/gossip/server/__init__.py b/src/gossip/server/__init__.py index f0881010..d7d5b576 100644 --- a/src/gossip/server/__init__.py +++ b/src/gossip/server/__init__.py @@ -39,8 +39,8 @@ inbound_dandelion_edge_count = [0] def gossip_server( - peer_set: OrderedSet['Peer'], - block_queues: Tuple[Queue['Block'], Queue['Block']], + peer_set: "OrderedSet[Peer]", + block_queues: Tuple["Queue[Block]", "Queue[Block]"], dandelion_seed: bytes): async def peer_connected( diff --git a/src/gossip/server/acceptstem.py b/src/gossip/server/acceptstem.py index fd528bee..a9225c18 100644 --- a/src/gossip/server/acceptstem.py +++ b/src/gossip/server/acceptstem.py @@ -20,7 +20,7 @@ if TYPE_CHECKING: async def accept_stem_blocks( - block_queues: Tuple[Queue['Block'], Queue['Block']], + block_queues: Tuple["Queue[Block]", "Queue[Block]"], reader: 'StreamReader', writer: 'StreamWriter', inbound_edge_count: List[int]):