This is an automated email from the ASF dual-hosted git repository.
adoroszlai pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ozone.git
The following commit(s) were added to refs/heads/master by this push:
new 0036c31448 HDDS-8010. Improve DN warning message when getBlock does
not find the block. (#4698)
0036c31448 is described below
commit 0036c314485038819dbfb30c8923b7ad83815d8d
Author: AARON0607 <[email protected]>
AuthorDate: Sun May 14 23:11:40 2023 +0800
HDDS-8010. Improve DN warning message when getBlock does not find the
block. (#4698)
---
.../hdds/scm/container/common/helpers/StorageContainerException.java | 1 -
.../apache/hadoop/ozone/container/common/helpers/ContainerUtils.java | 2 +-
.../apache/hadoop/ozone/container/keyvalue/impl/BlockManagerImpl.java | 4 ++--
3 files changed, 3 insertions(+), 4 deletions(-)
diff --git
a/hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/scm/container/common/helpers/StorageContainerException.java
b/hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/scm/container/common/helpers/StorageContainerException.java
index f2271be3bc..7bc87b6c66 100644
---
a/hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/scm/container/common/helpers/StorageContainerException.java
+++
b/hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/scm/container/common/helpers/StorageContainerException.java
@@ -100,5 +100,4 @@ public class StorageContainerException extends IOException {
return result;
}
-
}
diff --git
a/hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/common/helpers/ContainerUtils.java
b/hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/common/helpers/ContainerUtils.java
index 20bec12681..45a38c1618 100644
---
a/hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/common/helpers/ContainerUtils.java
+++
b/hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/common/helpers/ContainerUtils.java
@@ -88,7 +88,7 @@ public final class ContainerUtils {
ex.getMessage(), ex.getResult().getValueDescriptor().getName(),
ex);
}
} else {
- log.info(logInfo, request.getCmdType(), request.getTraceID(),
+ log.warn(logInfo, request.getCmdType(), request.getTraceID(),
ex.getMessage(), ex.getResult().getValueDescriptor().getName(), ex);
}
return getContainerCommandResponse(request, ex.getResult(),
ex.getMessage())
diff --git
a/hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/keyvalue/impl/BlockManagerImpl.java
b/hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/keyvalue/impl/BlockManagerImpl.java
index d274f0361f..f2c126451c 100644
---
a/hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/keyvalue/impl/BlockManagerImpl.java
+++
b/hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/keyvalue/impl/BlockManagerImpl.java
@@ -356,8 +356,8 @@ public class BlockManagerImpl implements BlockManager {
BlockData blockData = db.getStore().getBlockDataTable().get(blockKey);
if (blockData == null) {
- throw new StorageContainerException(NO_SUCH_BLOCK_ERR_MSG,
- NO_SUCH_BLOCK);
+ throw new StorageContainerException(NO_SUCH_BLOCK_ERR_MSG +
+ " BlockID : " + blockID, NO_SUCH_BLOCK);
}
return blockData;
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
