libp2p/py-libp2p

Medley of `pubsub` improvements

Aberta

#283 aberto em 5 de set. de 2019

Ver no GitHubGuia do colaborador
 (6 comentários) (1 reação) (0 responsável)Python (238 forks)auto 404
help wanted

Métricas do repositório

Stars
 (618 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

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

Guia do colaborador

Direção de pesquisa
Examine o envio serial de mensagens nas classes pubsub e refatore para usar asyncio.gather para concorrência. Em seguida, separe a seleção de pares (roteador) do envio de mensagens (Pubsub).
Pilha de tecnologia
python
Domain
backendnetworking
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Pythonasyncio
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60