only construct bitswap event loggable if necessary

Base58 encoding cids/peerIDs isn't exactly fast. License: MIT Signed-off-by: Steven Allen <steven@stebalien.com>
ipfs · Dec 29, 2017 · 914e653 · 914e653
1 parent 72714b6
commit 914e653
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/exchange/bitswap/workers.go b/exchange/bitswap/workers.go
@@ -59,11 +59,13 @@ func (bs *Bitswap) taskWorker(ctx context.Context, id int) {
 				if !ok {
 					continue
 				}
-				log.Event(ctx, "Bitswap.TaskWorker.Work", logging.LoggableMap{
-					"ID":     id,
-					"Target": envelope.Peer.Pretty(),
-					"Block":  envelope.Block.Cid().String(),
-				})
+				log.Event(ctx, "Bitswap.TaskWorker.Work", logging.LoggableF(func() map[string]interface{} {
+					return logging.LoggableMap{
+						"ID":     id,
+						"Target": envelope.Peer.Pretty(),
+						"Block":  envelope.Block.Cid().String(),
+					}
+				}))
 
 				// update the BS ledger to reflect sent message
 				// TODO: Should only track *useful* messages in ledger