akka/akka-core

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

Aperta

#21.100 aperta il 2 ago 2016

Vedi su GitHubGuida contributor
 (5 commenti) (0 reazioni) (0 assegnatari)Scala (3547 fork)batch import
1 - triagedhelp wantedt:cluster-tools

Metriche repository

Star
 (13.277 stelle)
Metriche merge PR
 (Merge medio 8g 19h) (10 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Esamina il file `akka cluster tools/src/main/scala/akka/cluster/pubsub/DistributedPubSubMediator.scala`, in particolare il metodo `prune()` intorno alla riga 812 e il tratto `PerGroupingBuffer`. Rivedi la suite di test esistente per `DistributedPubSub` sia nei test unitari che multi nodo. Studia la PR #19154 per comprendere le modifiche che necessitano di copertura dei test. Quindi, scrivi test per il comportamento di pruning e documenta la configurazione `removed time to live` nella documentazione di riferimento.
Tech stack
scala
Dominio
backendtesting
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
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
ScalaAkkatesting
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60