[jira] [Updated] (HBASE-12474) Incorrect handling of default namespace in user_permission command.

2014-11-20 Thread Srikanth Srungarapu (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-12474?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Srikanth Srungarapu updated HBASE-12474:

Attachment: HBASE-12474.patch

Matteo +1'ed HBASE-12073. This patch applies cleanly on top of it. 

> Incorrect handling of default namespace in user_permission command.
> ---
>
> Key: HBASE-12474
> URL: https://issues.apache.org/jira/browse/HBASE-12474
> Project: HBase
>  Issue Type: Bug
>Reporter: Srikanth Srungarapu
>Assignee: Srikanth Srungarapu
>Priority: Minor
> Attachments: HBASE-12474.patch
>
>
> The command user_permission returns 0 rows if we specify default prefix. It 
> works fine if we drop the prefix. This is because pattern matching of 
> listTables doesn't take into account that nameAsString method doesn't include 
> 'default' in it's output for the relevant tables. This method listTables is 
> also used by delete, disable and enable commands when invoked with regular 
> expression. Credits to [~mbertozzi] for coming up with this corner case.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-12474) Incorrect handling of default namespace in user_permission command.

2014-11-21 Thread Srikanth Srungarapu (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-12474?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Srikanth Srungarapu updated HBASE-12474:

Attachment: HBASE-12474_v2.patch

Addressing the following comments made by Matteo offline.
* Short circuiting the matched if true.
* Separate test for verifying the functionality with namespace, without 
namespace and default namespace.

> Incorrect handling of default namespace in user_permission command.
> ---
>
> Key: HBASE-12474
> URL: https://issues.apache.org/jira/browse/HBASE-12474
> Project: HBase
>  Issue Type: Bug
>Reporter: Srikanth Srungarapu
>Assignee: Srikanth Srungarapu
>Priority: Minor
> Attachments: HBASE-12474.patch, HBASE-12474_v2.patch
>
>
> The command user_permission returns 0 rows if we specify default prefix. It 
> works fine if we drop the prefix. This is because pattern matching of 
> listTables doesn't take into account that nameAsString method doesn't include 
> 'default' in it's output for the relevant tables. This method listTables is 
> also used by delete, disable and enable commands when invoked with regular 
> expression. Credits to [~mbertozzi] for coming up with this corner case.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-12474) Incorrect handling of default namespace in user_permission command.

2014-11-21 Thread Srikanth Srungarapu (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-12474?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Srikanth Srungarapu updated HBASE-12474:

Attachment: HBASE-12474_v2.patch

> Incorrect handling of default namespace in user_permission command.
> ---
>
> Key: HBASE-12474
> URL: https://issues.apache.org/jira/browse/HBASE-12474
> Project: HBase
>  Issue Type: Bug
>Reporter: Srikanth Srungarapu
>Assignee: Srikanth Srungarapu
>Priority: Minor
> Attachments: HBASE-12474.patch, HBASE-12474_v2.patch
>
>
> The command user_permission returns 0 rows if we specify default prefix. It 
> works fine if we drop the prefix. This is because pattern matching of 
> listTables doesn't take into account that nameAsString method doesn't include 
> 'default' in it's output for the relevant tables. This method listTables is 
> also used by delete, disable and enable commands when invoked with regular 
> expression. Credits to [~mbertozzi] for coming up with this corner case.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-12474) Incorrect handling of default namespace in user_permission command.

2014-11-21 Thread Srikanth Srungarapu (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-12474?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Srikanth Srungarapu updated HBASE-12474:

Attachment: (was: HBASE-12474_v2.patch)

> Incorrect handling of default namespace in user_permission command.
> ---
>
> Key: HBASE-12474
> URL: https://issues.apache.org/jira/browse/HBASE-12474
> Project: HBase
>  Issue Type: Bug
>Reporter: Srikanth Srungarapu
>Assignee: Srikanth Srungarapu
>Priority: Minor
> Attachments: HBASE-12474.patch, HBASE-12474_v2.patch
>
>
> The command user_permission returns 0 rows if we specify default prefix. It 
> works fine if we drop the prefix. This is because pattern matching of 
> listTables doesn't take into account that nameAsString method doesn't include 
> 'default' in it's output for the relevant tables. This method listTables is 
> also used by delete, disable and enable commands when invoked with regular 
> expression. Credits to [~mbertozzi] for coming up with this corner case.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-12474) Incorrect handling of default namespace in user_permission command.

2014-11-21 Thread Matteo Bertozzi (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-12474?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Matteo Bertozzi updated HBASE-12474:

Status: Patch Available  (was: Open)

> Incorrect handling of default namespace in user_permission command.
> ---
>
> Key: HBASE-12474
> URL: https://issues.apache.org/jira/browse/HBASE-12474
> Project: HBase
>  Issue Type: Bug
>Reporter: Srikanth Srungarapu
>Assignee: Srikanth Srungarapu
>Priority: Minor
> Attachments: HBASE-12474.patch, HBASE-12474_v2.patch
>
>
> The command user_permission returns 0 rows if we specify default prefix. It 
> works fine if we drop the prefix. This is because pattern matching of 
> listTables doesn't take into account that nameAsString method doesn't include 
> 'default' in it's output for the relevant tables. This method listTables is 
> also used by delete, disable and enable commands when invoked with regular 
> expression. Credits to [~mbertozzi] for coming up with this corner case.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-12474) Incorrect handling of default namespace in user_permission command.

2014-11-21 Thread Matteo Bertozzi (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-12474?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Matteo Bertozzi updated HBASE-12474:

Status: Open  (was: Patch Available)

> Incorrect handling of default namespace in user_permission command.
> ---
>
> Key: HBASE-12474
> URL: https://issues.apache.org/jira/browse/HBASE-12474
> Project: HBase
>  Issue Type: Bug
>Reporter: Srikanth Srungarapu
>Assignee: Srikanth Srungarapu
>Priority: Minor
> Attachments: HBASE-12474.patch, HBASE-12474_v2.patch
>
>
> The command user_permission returns 0 rows if we specify default prefix. It 
> works fine if we drop the prefix. This is because pattern matching of 
> listTables doesn't take into account that nameAsString method doesn't include 
> 'default' in it's output for the relevant tables. This method listTables is 
> also used by delete, disable and enable commands when invoked with regular 
> expression. Credits to [~mbertozzi] for coming up with this corner case.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-12474) Incorrect handling of default namespace in user_permission command.

2014-11-21 Thread Matteo Bertozzi (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-12474?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Matteo Bertozzi updated HBASE-12474:

Status: Patch Available  (was: Open)

> Incorrect handling of default namespace in user_permission command.
> ---
>
> Key: HBASE-12474
> URL: https://issues.apache.org/jira/browse/HBASE-12474
> Project: HBase
>  Issue Type: Bug
>Reporter: Srikanth Srungarapu
>Assignee: Srikanth Srungarapu
>Priority: Minor
> Attachments: HBASE-12474.patch, HBASE-12474_v2.patch
>
>
> The command user_permission returns 0 rows if we specify default prefix. It 
> works fine if we drop the prefix. This is because pattern matching of 
> listTables doesn't take into account that nameAsString method doesn't include 
> 'default' in it's output for the relevant tables. This method listTables is 
> also used by delete, disable and enable commands when invoked with regular 
> expression. Credits to [~mbertozzi] for coming up with this corner case.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-12474) Incorrect handling of default namespace in user_permission command.

2014-11-21 Thread Srikanth Srungarapu (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-12474?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Srikanth Srungarapu updated HBASE-12474:

Attachment: HBASE-12474_v2.patch

Re-attaching the patch for Hadoop QA to pick up.

> Incorrect handling of default namespace in user_permission command.
> ---
>
> Key: HBASE-12474
> URL: https://issues.apache.org/jira/browse/HBASE-12474
> Project: HBase
>  Issue Type: Bug
>Reporter: Srikanth Srungarapu
>Assignee: Srikanth Srungarapu
>Priority: Minor
> Attachments: HBASE-12474.patch, HBASE-12474_v2.patch
>
>
> The command user_permission returns 0 rows if we specify default prefix. It 
> works fine if we drop the prefix. This is because pattern matching of 
> listTables doesn't take into account that nameAsString method doesn't include 
> 'default' in it's output for the relevant tables. This method listTables is 
> also used by delete, disable and enable commands when invoked with regular 
> expression. Credits to [~mbertozzi] for coming up with this corner case.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-12474) Incorrect handling of default namespace in user_permission command.

2014-11-21 Thread Srikanth Srungarapu (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-12474?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Srikanth Srungarapu updated HBASE-12474:

Attachment: (was: HBASE-12474_v2.patch)

> Incorrect handling of default namespace in user_permission command.
> ---
>
> Key: HBASE-12474
> URL: https://issues.apache.org/jira/browse/HBASE-12474
> Project: HBase
>  Issue Type: Bug
>Reporter: Srikanth Srungarapu
>Assignee: Srikanth Srungarapu
>Priority: Minor
> Attachments: HBASE-12474.patch, HBASE-12474_v2.patch
>
>
> The command user_permission returns 0 rows if we specify default prefix. It 
> works fine if we drop the prefix. This is because pattern matching of 
> listTables doesn't take into account that nameAsString method doesn't include 
> 'default' in it's output for the relevant tables. This method listTables is 
> also used by delete, disable and enable commands when invoked with regular 
> expression. Credits to [~mbertozzi] for coming up with this corner case.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-12474) Incorrect handling of default namespace in user_permission command.

2014-11-24 Thread Matteo Bertozzi (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-12474?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Matteo Bertozzi updated HBASE-12474:

Attachment: HBASE-12474-v3.patch

the test was flaky because dependent on the execution order 
testAccessControllerRegexHandling() and testGetNamespacePermission() are   
creating "testNamespace"... but the new testAccessControllerRegexHandling() was 
not removing it at the end of the test. (v3 removes the created namespace 
tables)

> Incorrect handling of default namespace in user_permission command.
> ---
>
> Key: HBASE-12474
> URL: https://issues.apache.org/jira/browse/HBASE-12474
> Project: HBase
>  Issue Type: Bug
>Reporter: Srikanth Srungarapu
>Assignee: Srikanth Srungarapu
>Priority: Minor
> Attachments: HBASE-12474-v3.patch, HBASE-12474.patch, 
> HBASE-12474_v2.patch
>
>
> The command user_permission returns 0 rows if we specify default prefix. It 
> works fine if we drop the prefix. This is because pattern matching of 
> listTables doesn't take into account that nameAsString method doesn't include 
> 'default' in it's output for the relevant tables. This method listTables is 
> also used by delete, disable and enable commands when invoked with regular 
> expression. Credits to [~mbertozzi] for coming up with this corner case.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Updated] (HBASE-12474) Incorrect handling of default namespace in user_permission command.

2014-11-26 Thread Matteo Bertozzi (JIRA)

 [ 
https://issues.apache.org/jira/browse/HBASE-12474?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Matteo Bertozzi updated HBASE-12474:

   Resolution: Fixed
Fix Version/s: 2.0.0
   Status: Resolved  (was: Patch Available)

> Incorrect handling of default namespace in user_permission command.
> ---
>
> Key: HBASE-12474
> URL: https://issues.apache.org/jira/browse/HBASE-12474
> Project: HBase
>  Issue Type: Bug
>Reporter: Srikanth Srungarapu
>Assignee: Srikanth Srungarapu
>Priority: Minor
> Fix For: 2.0.0
>
> Attachments: HBASE-12474-v3.patch, HBASE-12474.patch, 
> HBASE-12474_v2.patch
>
>
> The command user_permission returns 0 rows if we specify default prefix. It 
> works fine if we drop the prefix. This is because pattern matching of 
> listTables doesn't take into account that nameAsString method doesn't include 
> 'default' in it's output for the relevant tables. This method listTables is 
> also used by delete, disable and enable commands when invoked with regular 
> expression. Credits to [~mbertozzi] for coming up with this corner case.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)