nsqio/nsq

nsqd: statistics aren't recorded when client publishes to multiple topics

Open

#1492 aperta il 5 lug 2024

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Go (2898 fork)batch import
bughelp wanted

Metriche repository

Star
 (25.788 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

When a client sends messages to multiple topics, the statistics information can only output the results of one topic for a single client. After reviewing the code, I found that the problem lies in the break statement on line 305 of the client_v2.go file. Can you explain why this break statement is added?

for topic, count := range c.pubCounts {
	if len(topicName) > 0 && topic != topicName {
		continue
	}
	pubCounts = append(pubCounts, PubCount{
		Topic: topic,
		Count: count,
	})
	break   **//  What does this break statement do?**
}

Guida contributor

Direzione di ricerca
Traccia il percorso del codice in client v2.go dove viene costruito pubCounts e identifica come l'istruzione break impedisce l'accumulo dei conteggi per più topic.
Tech stack
go
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
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
Go
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85