[
https://issues.apache.org/jira/browse/HDDS-1686?focusedWorklogId=263845&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-263845
]
ASF GitHub Bot logged work on HDDS-1686:
----------------------------------------
Author: ASF GitHub Bot
Created on: 20/Jun/19 15:24
Start Date: 20/Jun/19 15:24
Worklog Time Spent: 10m
Work Description: bharatviswa504 commented on pull request #966:
HDDS-1686. Remove check to get from openKeyTable in acl implementatio…
URL: https://github.com/apache/hadoop/pull/966#discussion_r295861790
##########
File path:
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/KeyManagerImpl.java
##########
@@ -1370,17 +1370,10 @@ public boolean addAcl(OzoneObj obj, OzoneAcl acl)
throws IOException {
validateBucket(volume, bucket);
String objectKey = metadataManager.getOzoneKey(volume, bucket, keyName);
OmKeyInfo keyInfo = metadataManager.getKeyTable().get(objectKey);
- Table keyTable;
if (keyInfo == null) {
Review comment:
@xiaoyuyao Sure we can do that. What is the use case for setting acl for an
openKey which is not committed to OM. Just want to understand the reason behind
it.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Issue Time Tracking
-------------------
Worklog Id: (was: 263845)
Time Spent: 50m (was: 40m)
> Remove check to get from openKeyTable in acl implementation for Keys
> --------------------------------------------------------------------
>
> Key: HDDS-1686
> URL: https://issues.apache.org/jira/browse/HDDS-1686
> Project: Hadoop Distributed Data Store
> Issue Type: Sub-task
> Reporter: Bharat Viswanadham
> Assignee: Bharat Viswanadham
> Priority: Major
> Labels: pull-request-available
> Time Spent: 50m
> Remaining Estimate: 0h
>
> Remove the checks for openKeyTable in acl Implementation for Keys.
> As entries in openKeyTable are appended with clientID, so the entry will not
> be found in openKey table 100% of the time. We can save 1 DB read in this way.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org
