[jira] [Commented] (HBASE-10970) [AccessController] delete latest version should check covering permission for the latest version only

2014-04-23 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13977952#comment-13977952 ] ramkrishna.s.vasudevan commented on HBASE-10970: I went thro the patch.

[jira] [Commented] (HBASE-10970) [AccessController] delete latest version should check covering permission for the latest version only

2014-04-23 Thread Anoop Sam John (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13977959#comment-13977959 ] Anoop Sam John commented on HBASE-10970: bq.But I still feel atleast for Put

[jira] [Commented] (HBASE-10970) [AccessController] delete latest version should check covering permission for the latest version only

2014-04-23 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13977990#comment-13977990 ] ramkrishna.s.vasudevan commented on HBASE-10970: bq.This is a special

[jira] [Commented] (HBASE-10970) [AccessController] delete latest version should check covering permission for the latest version only

2014-04-23 Thread Anoop Sam John (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13977993#comment-13977993 ] Anoop Sam John commented on HBASE-10970: bq. For Put it may not be really needed

[jira] [Commented] (HBASE-10970) [AccessController] delete latest version should check covering permission for the latest version only

2014-04-23 Thread Anoop Sam John (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13977997#comment-13977997 ] Anoop Sam John commented on HBASE-10970: Understood ur concern on the max

[jira] [Commented] (HBASE-10970) [AccessController] delete latest version should check covering permission for the latest version only

2014-04-23 Thread Anoop Sam John (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13978315#comment-13978315 ] Anoop Sam John commented on HBASE-10970: There is one optimization possible ,

[jira] [Commented] (HBASE-10970) [AccessController] delete latest version should check covering permission for the latest version only

2014-04-23 Thread Andrew Purtell (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13978484#comment-13978484 ] Andrew Purtell commented on HBASE-10970: Made another pass over this. Still looks

[jira] [Commented] (HBASE-10970) [AccessController] delete latest version should check covering permission for the latest version only

2014-04-23 Thread Anoop Sam John (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13978491#comment-13978491 ] Anoop Sam John commented on HBASE-10970: bq.So why not just use List here? Sure

[jira] [Commented] (HBASE-10970) [AccessController] delete latest version should check covering permission for the latest version only

2014-04-23 Thread Andrew Purtell (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13978523#comment-13978523 ] Andrew Purtell commented on HBASE-10970: +1 [AccessController] delete latest

[jira] [Commented] (HBASE-10970) [AccessController] delete latest version should check covering permission for the latest version only

2014-04-23 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13978843#comment-13978843 ] Hadoop QA commented on HBASE-10970: --- {color:green}+1 overall{color}. Here are the

[jira] [Commented] (HBASE-10970) [AccessController] delete latest version should check covering permission for the latest version only

2014-04-23 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13979284#comment-13979284 ] ramkrishna.s.vasudevan commented on HBASE-10970: +1 on v3.

[jira] [Commented] (HBASE-10970) [AccessController] delete latest version should check covering permission for the latest version only

2014-04-22 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13976683#comment-13976683 ] Hadoop QA commented on HBASE-10970: --- {color:green}+1 overall{color}. Here are the

[jira] [Commented] (HBASE-10970) [AccessController] delete latest version should check covering permission for the latest version only

2014-04-22 Thread Andrew Purtell (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13977004#comment-13977004 ] Andrew Purtell commented on HBASE-10970: Ok well consensus seems to be do it this

[jira] [Commented] (HBASE-10970) [AccessController] delete latest version should check covering permission for the latest version only

2014-04-15 Thread Andrew Purtell (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13969662#comment-13969662 ] Andrew Purtell commented on HBASE-10970: Why not one fix to take care of the

[jira] [Commented] (HBASE-10970) [AccessController] delete latest version should check covering permission for the latest version only

2014-04-15 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13969699#comment-13969699 ] ramkrishna.s.vasudevan commented on HBASE-10970: bq.Why not one fix to

[jira] [Commented] (HBASE-10970) [AccessController] delete latest version should check covering permission for the latest version only

2014-04-14 Thread Andrew Purtell (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13969123#comment-13969123 ] Andrew Purtell commented on HBASE-10970: This issue doesn't say what the

[jira] [Commented] (HBASE-10970) [AccessController] delete latest version should check covering permission for the latest version only

2014-04-14 Thread Andrew Purtell (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13969124#comment-13969124 ] Andrew Purtell commented on HBASE-10970: For example, compare this issue with the

[jira] [Commented] (HBASE-10970) [AccessController] delete latest version should check covering permission for the latest version only

2014-04-14 Thread ramkrishna.s.vasudevan (JIRA)
[ https://issues.apache.org/jira/browse/HBASE-10970?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13969206#comment-13969206 ] ramkrishna.s.vasudevan commented on HBASE-10970: [~apurtell] This JIRA i