ctrezzo commented on code in PR #6598:
URL: https://github.com/apache/hadoop/pull/6598#discussion_r1509499856
##########
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/security/token/block/BlockTokenSecretManager.java:
##########
@@ -226,7 +228,7 @@ public synchronized void addKeys(ExportedBlockKeys
exportedKeys)
if (isMaster || exportedKeys == null) {
return;
}
- LOG.info("Setting block keys. BlockPool = {} .", blockPoolId);
+ LOG.info("Setting block keys. BlockPool = {} . NamenodeIndex = {} .",
blockPoolId, nnIndex);
Review Comment:
I don't think nnIndex is what you want here. If you are looking for
information around which nn this request came from, I would instead pass the
BPServiceActor in from the BPOfferService. Then you can print out the nnAddr
and the HAServiceState along with the log message.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
