libp2p/py-libp2p

Medley of `pubsub` improvements

Aperta

#283 aperta il 5 set 2019

Vedi su GitHubGuida contributor
 (6 commenti) (1 reazione) (0 assegnatari)Python (238 fork)auto 404
help wanted

Metriche repository

Star
 (618 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Leaving this here while I'm reviewing the current pubsub packages...

  1. Don't serialize message sends, eg: https://github.com/libp2p/py-libp2p/blob/master/libp2p/pubsub/gossipsub.py#L172-L177 Happens here and in floodsub as well -- this is a great place to use asyncio.gather

  2. "separate policy from mechanism" -- in the same place, we defer to the particular router to decide who to send new msgs to... and then do the sending in the router itself. as far as i can tell, we can pull out the sending part into the Pubsub class, relying on the router for what should be its only job -- determining peers for some action

Guida contributor

Direzione di ricerca
Esamina l'invio seriale dei messaggi nelle classi pubsub e rifattorizza per utilizzare asyncio.gather per la concorrenza. Quindi separa la selezione dei peer (router) dall'invio dei messaggi (Pubsub).
Tech stack
python
Dominio
backendnetworking
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Pythonasyncio
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60