Mailing-List: contact issues-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Date: Wed, 23 Apr 2014 17:19:16 +0000 (UTC)
From: "Andrew Purtell (JIRA)" <jira@apache.org>
To: issues@hbase.apache.org
Message-ID: <JIRA.12708182.1397449319651.164771.1398273556527@arcas>
In-Reply-To: <JIRA.12708182.1397449319651@arcas>
References: <JIRA.12708182.1397449319651@arcas>
Subject: [jira] [Commented] (HBASE-10970) [AccessController] delete latest
 version should check covering permission for the latest  version only
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13978484#comment-13978484 ] 

Andrew Purtell commented on HBASE-10970:
----------------------------------------

Made another pass over this. Still looks like an improvement so I am still +1. One minor question:

{code}
+      Map<ByteRange, Object> familyMap1 = new HashMap<ByteRange, Object>();
{code}

and below the code only considers List. So why not just use List here? 

> [AccessController] delete latest version should check covering permission for the latest  version only
> ------------------------------------------------------------------------------------------------------
>
>                 Key: HBASE-10970
>                 URL: https://issues.apache.org/jira/browse/HBASE-10970
>             Project: HBase
>          Issue Type: Bug
>    Affects Versions: 0.98.0
>            Reporter: ramkrishna.s.vasudevan
>            Assignee: Anoop Sam John
>             Fix For: 0.99.0, 0.98.2
>
>         Attachments: HBASE-10970.patch, HBASE-10970_V2.patch
>
>
> Refer to discussion in HBASE-10823.
> bq.The exact version deletion also check for the preceeding version though the latest version allows the permission


--
This message was sent by Atlassian JIRA
(v6.2#6252)