From ce0761050eda04631ae437d32b8bed2e4169b4c4 Mon Sep 17 00:00:00 2001 From: Seebs Date: Fri, 3 Apr 2020 13:06:06 -0500 Subject: [PATCH] less spammy snapshotqueue During testing we spawn a lot of tiny snapshot queues. Make the message less spammy by printing it only if any enqueues were skipped (shouldn't ever happen) or more than one thing got enqueued (likely in real usage, but doesn't happen in testing usually). --- snapshotqueue.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/snapshotqueue.go b/snapshotqueue.go index 9df79e26d..373e5c538 100644 --- a/snapshotqueue.go +++ b/snapshotqueue.go @@ -192,7 +192,9 @@ func (sq *prioritySnapshotQueue) Stop() { sq.urgent = nil close(sq.background) sq.background = nil - sq.logger.Printf("snapshot queue: enqueued %d, skipped %d\n", sq.stats.enqueued, sq.stats.skipped) + if sq.stats.skipped > 0 || sq.stats.enqueued > 1 { + sq.logger.Printf("snapshot queue: enqueued %d, skipped %d\n", sq.stats.enqueued, sq.stats.skipped) + } } // Enqueue tries to add a fragment to the queue, if the fragment is not already