You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by ae...@apache.org on 2019/07/22 22:47:12 UTC
[hadoop] 01/02: HDDS-1686. Remove check to get from openKeyTable in acl implementatio… (#966)
This is an automated email from the ASF dual-hosted git repository.
aengineer pushed a commit to branch ozone-0.4.1
in repository https://gitbox.apache.org/repos/asf/hadoop.git
commit 9108efa8afbee80420c76c625237af22dd4fab84
Author: Bharat Viswanadham <bh...@apache.org>
AuthorDate: Mon Jul 22 15:11:10 2019 -0700
HDDS-1686. Remove check to get from openKeyTable in acl implementatio… (#966)
(cherry picked from commit 2ea71d953b46221f90b38d75a2999056f044471f)
---
.../org/apache/hadoop/ozone/om/KeyManagerImpl.java | 43 ++++------------------
1 file changed, 8 insertions(+), 35 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 9b4eac3..ce55210 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
@@ -1398,17 +1398,10 @@ public class KeyManagerImpl implements KeyManager {
validateBucket(volume, bucket);
String objectKey = metadataManager.getOzoneKey(volume, bucket, keyName);
OmKeyInfo keyInfo = metadataManager.getKeyTable().get(objectKey);
- Table keyTable;
if (keyInfo == null) {
- keyInfo = metadataManager.getOpenKeyTable().get(objectKey);
- if (keyInfo == null) {
- throw new OMException("Key not found. Key:" +
- objectKey, KEY_NOT_FOUND);
- }
- keyTable = metadataManager.getOpenKeyTable();
- } else {
- keyTable = metadataManager.getKeyTable();
+ throw new OMException("Key not found. Key:" + objectKey, KEY_NOT_FOUND);
}
+
List<OzoneAclInfo> newAcls = new ArrayList<>(keyInfo.getAcls());
OzoneAclInfo newAcl = null;
for(OzoneAclInfo a: keyInfo.getAcls()) {
@@ -1444,7 +1437,7 @@ public class KeyManagerImpl implements KeyManager {
.setDataSize(keyInfo.getDataSize())
.setFileEncryptionInfo(keyInfo.getFileEncryptionInfo())
.build();
- keyTable.put(objectKey, newObj);
+ metadataManager.getKeyTable().put(objectKey, newObj);
} catch (IOException ex) {
if (!(ex instanceof OMException)) {
LOG.error("Add acl operation failed for key:{}/{}/{}", volume,
@@ -1477,16 +1470,8 @@ public class KeyManagerImpl implements KeyManager {
validateBucket(volume, bucket);
String objectKey = metadataManager.getOzoneKey(volume, bucket, keyName);
OmKeyInfo keyInfo = metadataManager.getKeyTable().get(objectKey);
- Table keyTable;
if (keyInfo == null) {
- keyInfo = metadataManager.getOpenKeyTable().get(objectKey);
- if (keyInfo == null) {
- throw new OMException("Key not found. Key:" +
- objectKey, KEY_NOT_FOUND);
- }
- keyTable = metadataManager.getOpenKeyTable();
- } else {
- keyTable = metadataManager.getKeyTable();
+ throw new OMException("Key not found. Key:" + objectKey, KEY_NOT_FOUND);
}
List<OzoneAclInfo> newAcls = new ArrayList<>(keyInfo.getAcls());
@@ -1531,7 +1516,7 @@ public class KeyManagerImpl implements KeyManager {
.setFileEncryptionInfo(keyInfo.getFileEncryptionInfo())
.build();
- keyTable.put(objectKey, newObj);
+ metadataManager.getKeyTable().put(objectKey, newObj);
} catch (IOException ex) {
if (!(ex instanceof OMException)) {
LOG.error("Remove acl operation failed for key:{}/{}/{}", volume,
@@ -1564,16 +1549,8 @@ public class KeyManagerImpl implements KeyManager {
validateBucket(volume, bucket);
String objectKey = metadataManager.getOzoneKey(volume, bucket, keyName);
OmKeyInfo keyInfo = metadataManager.getKeyTable().get(objectKey);
- Table keyTable;
if (keyInfo == null) {
- keyInfo = metadataManager.getOpenKeyTable().get(objectKey);
- if (keyInfo == null) {
- throw new OMException("Key not found. Key:" +
- objectKey, KEY_NOT_FOUND);
- }
- keyTable = metadataManager.getOpenKeyTable();
- } else {
- keyTable = metadataManager.getKeyTable();
+ throw new OMException("Key not found. Key:" + objectKey, KEY_NOT_FOUND);
}
List<OzoneAclInfo> newAcls = new ArrayList<>();
@@ -1594,7 +1571,7 @@ public class KeyManagerImpl implements KeyManager {
.setFileEncryptionInfo(keyInfo.getFileEncryptionInfo())
.build();
- keyTable.put(objectKey, newObj);
+ metadataManager.getKeyTable().put(objectKey, newObj);
} catch (IOException ex) {
if (!(ex instanceof OMException)) {
LOG.error("Set acl operation failed for key:{}/{}/{}", volume,
@@ -1626,11 +1603,7 @@ public class KeyManagerImpl implements KeyManager {
String objectKey = metadataManager.getOzoneKey(volume, bucket, keyName);
OmKeyInfo keyInfo = metadataManager.getKeyTable().get(objectKey);
if (keyInfo == null) {
- keyInfo = metadataManager.getOpenKeyTable().get(objectKey);
- if (keyInfo == null) {
- throw new OMException("Key not found. Key:" +
- objectKey, KEY_NOT_FOUND);
- }
+ throw new OMException("Key not found. Key:" + objectKey, KEY_NOT_FOUND);
}
List<OzoneAcl> acls = new ArrayList<>();
---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org