From 4b4d6483044fe1f71729e26ef482d85bdb42803f Mon Sep 17 00:00:00 2001
From: Daniel Liu <139250065@qq.com>
Date: Tue, 11 Aug 2026 08:09:17 +0800
Subject: [PATCH] refactor(consensus,core): simplify masternode logging

Consolidate masternode log output into the engine update paths.
Remove the redundant candidate dump from block processing and keep the snapshot update logs focused on node address and stake.
---
 consensus/XDPoS/engines/engine_v1/engine.go | 4 ++--
 consensus/XDPoS/engines/engine_v2/engine.go | 2 +-
 core/blockchain.go                          | 7 -------
 3 files changed, 3 insertions(+), 10 deletions(-)

diff --git a/consensus/XDPoS/engines/engine_v1/engine.go b/consensus/XDPoS/engines/engine_v1/engine.go
index 57f0e4efcdb2..82e22d494b7f 100644
--- a/consensus/XDPoS/engines/engine_v1/engine.go
+++ b/consensus/XDPoS/engines/engine_v1/engine.go
@@ -832,8 +832,8 @@ func (x *XDPoS_v1) UpdateMasternodes(chain consensus.ChainReader, header *types.
 	}
 	x.recents.Add(snap.Hash, snap)
 	log.Info("New set of masternodes has been updated to snapshot", "number", snap.Number, "hash", snap.Hash)
-	for i, v := range nm {
-		log.Info("masternodes", "i", i, "addr", v)
+	for i, n := range ms {
+		log.Info("node", "i", i, "addr", n.Address, "stake", n.Stake)
 	}
 	return nil
 }
diff --git a/consensus/XDPoS/engines/engine_v2/engine.go b/consensus/XDPoS/engines/engine_v2/engine.go
index 854b16ff8939..331071a1569b 100644
--- a/consensus/XDPoS/engines/engine_v2/engine.go
+++ b/consensus/XDPoS/engines/engine_v2/engine.go
@@ -634,7 +634,7 @@ func (x *XDPoS_v2) UpdateMasternodes(chain consensus.ChainReader, header *types.
 
 	log.Info("[UpdateMasternodes] New set of masternodes has been updated to snapshot", "number", snap.Number, "hash", snap.Hash)
 	for i, n := range ms {
-		log.Info("masternode", "index", i, "address", n.Address)
+		log.Info("node", "i", i, "addr", n.Address, "stake", n.Stake)
 	}
 
 	return nil
diff --git a/core/blockchain.go b/core/blockchain.go
index ca337fb42a4f..54f5291c8895 100644
--- a/core/blockchain.go
+++ b/core/blockchain.go
@@ -2812,14 +2812,7 @@ func (bc *BlockChain) UpdateM1() error {
 		xdc_sort.Slice(ms, func(i, j int) bool {
 			return ms[i].Stake.Cmp(ms[j].Stake) >= 0
 		})
-		log.Info("Ordered list of masternode candidates")
-		for _, m := range ms {
-			log.Info("", "address", m.Address, "stake", m.Stake)
-		}
-		// update masternodes
-
 		log.Info("Updating new set of masternodes")
-		// get block header
 		header := bc.CurrentHeader()
 		err = engine.UpdateMasternodes(bc, header, ms)
 		if err != nil {