From 24cca2f18d4a7abb103179a289db09c2d11bd2a3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?P=C3=A9ter=20Szil=C3=A1gyi?= Date: Thu, 4 Jun 2015 15:10:43 +0300 Subject: [PATCH] eth/downloader: log after state updates, easier to debug --- eth/downloader/downloader.go | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/eth/downloader/downloader.go b/eth/downloader/downloader.go index 806f60f1b..d8dbef726 100644 --- a/eth/downloader/downloader.go +++ b/eth/downloader/downloader.go @@ -428,24 +428,22 @@ out: return err } // Peer did deliver, but some blocks were off, penalize - glog.V(logger.Detail).Infof("%s: block delivery failed: %v", peer, err) peer.Demote() peer.SetIdle() + glog.V(logger.Detail).Infof("%s: block delivery failed: %v", peer, err) break } // If no blocks were delivered, demote the peer (above code is needed to mark the packet done!) if len(blockPack.blocks) == 0 { - glog.V(logger.Detail).Infof("%s: no blocks delivered", peer) peer.Demote() peer.SetIdle() + glog.V(logger.Detail).Infof("%s: no blocks delivered", peer) break } // All was successful, promote the peer - if glog.V(logger.Detail) && len(blockPack.blocks) > 0 { - glog.Infof("%s: delivered %d blocks", peer, len(blockPack.blocks)) - } peer.Promote() peer.SetIdle() + glog.V(logger.Detail).Infof("%s: delivered %d blocks", peer, len(blockPack.blocks)) } case <-ticker.C: // Check for bad peers. Bad peers may indicate a peer not responding