apache · adoroszlai · May 14, 2023 · Mar 6, 2023 · Mar 20, 2023 · Mar 21, 2023
diff --git a/...n/java/org/apache/hadoop/hdds/scm/container/common/helpers/StorageContainerException.java b/...n/java/org/apache/hadoop/hdds/scm/container/common/helpers/StorageContainerException.java
@@ -100,5 +100,4 @@ public ContainerProtos.Result getResult() {
     return result;
   }
 
-
 }
diff --git a/...ervice/src/main/java/org/apache/hadoop/ozone/container/common/helpers/ContainerUtils.java b/...ervice/src/main/java/org/apache/hadoop/ozone/container/common/helpers/ContainerUtils.java
@@ -88,7 +88,7 @@ public static ContainerCommandResponseProto logAndReturnError(
             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/...rvice/src/main/java/org/apache/hadoop/ozone/container/keyvalue/impl/BlockManagerImpl.java b/...rvice/src/main/java/org/apache/hadoop/ozone/container/keyvalue/impl/BlockManagerImpl.java
@@ -356,8 +356,8 @@ private BlockData getBlockByID(DBHandle db, BlockID blockID,
 
     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;
Original file line number	Diff line number	Diff line change
Expand Up		@@ -100,5 +100,4 @@ public ContainerProtos.Result getResult() {
		return result;
		}


Copy link Contributor adoroszlai May 11, 2023 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. Normally we try to avoid whitespace-only changes in unrelated files. In order to save CI time, I'm OK with it, unless someone requests other changes, too. Copy link Contributor Author GAOJHIHCYUAN May 15, 2023 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. @adoroszlai Thanks your reminding, I will notice next time.
		}