big-data/hadoop
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

HDFS-16391. Avoid evaluation of LOG.debug statement in NameNodeHeartbeatService (#3820)

Browse Source 
(cherry picked from commit 8d251bd629)
This commit is contained in:
wangzhaohui 2021-12-21 19:28:57 +08:00 committed by S O'Donnell
parent a4496a0187
commit cc7b7e174c
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
hadoop-hdfs-project/hadoop-hdfs-rbf/src/main/java/org/apache/hadoop/hdfs/server/federation/router/NamenodeHeartbeatService.java
View File

@ -221,12 +221,16 @@ private void updateState() {
LOG.error("Namenode is not operational: {}", getNamenodeDesc()); LOG.error("Namenode is not operational: {}", getNamenodeDesc());
} else if (report.haStateValid()) { } else if (report.haStateValid()) {
// block and HA status available // block and HA status available
LOG.debug("Received service state: {} from HA namenode: {}", if (LOG.isDebugEnabled()) {
report.getState(), getNamenodeDesc()); LOG.debug("Received service state: {} from HA namenode: {}",
report.getState(), getNamenodeDesc());
}
} else if (localTarget == null) { } else if (localTarget == null) {
// block info available, HA status not expected // block info available, HA status not expected
LOG.debug( if (LOG.isDebugEnabled()) {
"Reporting non-HA namenode as operational: " + getNamenodeDesc()); LOG.debug(
"Reporting non-HA namenode as operational: {}", getNamenodeDesc());
}
} else { } else {
// block info available, HA status should be available, but was not // block info available, HA status should be available, but was not
// fetched do nothing and let the current state stand // fetched do nothing and let the current state stand