akka/akka-core

Pruning in DistributedPubSub is not covered by tests and in the documentation

Aberta

#21.100 aberto em 2 de ago. de 2016

Ver no GitHubGuia do colaborador
 (5 comentários) (0 reação) (0 responsável)Scala (3.547 forks)batch import
1 - triagedhelp wantedt:cluster-tools

Métricas do repositório

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

Description

I've tried to understand, how removed-time-to-live works in DistributedPub sub, but I haven't found any references in the docs. Also the method prune() https://github.com/akka/akka/blob/master/akka-cluster-tools/src/main/scala/akka/cluster/pubsub/DistributedPubSubMediator.scala#L812 is not covered neither by unittests nor by multi-node tests. Also you have implemented a trait "PerGroupingBuffer" in 2.4.2, but his trait also is not covered by tests. It would be good to have tests for the whole PR: https://github.com/akka/akka/pull/19154

Guia do colaborador

Direção de pesquisa
Examine o arquivo `akka cluster tools/src/main/scala/akka/cluster/pubsub/DistributedPubSubMediator.scala`, especificamente o método `prune()` por volta da linha 812 e a trait `PerGroupingBuffer`. Revise a suíte de testes existente para `DistributedPubSub` em testes unitários e multi nó. Estude o PR #19154 para entender as mudanças que precisam de cobertura de testes. Em seguida, escreva testes para o comportamento de poda (pruning) e documente a configuração `removed time to live` na documentação de referência.
Pilha de tecnologia
scala
Domain
backendtesting
Tipo Issue
Teste
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.
1-2 dias
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
ScalaAkkatesting
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60