You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ozone.apache.org by ad...@apache.org on 2023/04/26 08:22:30 UTC
[ozone] branch master updated: HDDS-8475. Unexpected IOException should be returned as INTERNAL_ERROR. (#4609)
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 e50e0a6321 HDDS-8475. Unexpected IOException should be returned as INTERNAL_ERROR. (#4609)
e50e0a6321 is described below
commit e50e0a63211efaf1bbbe0291f80980f59b934aef
Author: Duong Nguyen <du...@gmail.com>
AuthorDate: Wed Apr 26 01:22:23 2023 -0700
HDDS-8475. Unexpected IOException should be returned as INTERNAL_ERROR. (#4609)
---
.../main/java/org/apache/hadoop/ozone/om/KeyManagerImpl.java | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/KeyManagerImpl.java b/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/KeyManagerImpl.java
index 75e7500429..327a918898 100644
--- a/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/KeyManagerImpl.java
+++ b/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/KeyManagerImpl.java
@@ -101,6 +101,7 @@ import com.google.common.base.Preconditions;
import com.google.common.base.Strings;
import org.apache.commons.lang3.StringUtils;
+import static java.lang.String.format;
import static org.apache.hadoop.fs.CommonConfigurationKeysPublic.HADOOP_SECURITY_KEY_PROVIDER_PATH;
import static org.apache.hadoop.hdds.HddsConfigKeys.HDDS_BLOCK_TOKEN_ENABLED;
import static org.apache.hadoop.hdds.HddsConfigKeys.HDDS_BLOCK_TOKEN_ENABLED_DEFAULT;
@@ -413,11 +414,10 @@ public class KeyManagerImpl implements KeyManager {
if (ex instanceof OMException) {
throw ex;
}
- if (LOG.isDebugEnabled()) {
- LOG.debug("Get key failed for volume:{} bucket:{} key:{}", volumeName,
- bucketName, keyName, ex);
- }
- throw new OMException(ex.getMessage(), KEY_NOT_FOUND);
+ throw new OMException(
+ format("Error reading key metadata: /%s/%s/%s",
+ volumeName, bucketName, keyName),
+ ex, INTERNAL_ERROR);
} finally {
metadataManager.getLock().releaseReadLock(BUCKET_LOCK, volumeName,
bucketName);
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@ozone.apache.org
For additional commands, e-mail: commits-help@ozone.apache.org