[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16782310#comment-16782310
]
Reid Chan commented on HBASE-21481:
---
Thanks! [~zghaobac]
> [acl] Superuser's permissions should not
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16782301#comment-16782301
]
Hudson commented on HBASE-21481:
Results for branch master
[build #832 on
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16782202#comment-16782202
]
Hudson commented on HBASE-21481:
Results for branch branch-2
[build #1720 on
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16782150#comment-16782150
]
Hudson commented on HBASE-21481:
Results for branch branch-2.2
[build #75 on
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16781603#comment-16781603
]
Hadoop QA commented on HBASE-21481:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16781523#comment-16781523
]
Guanghao Zhang commented on HBASE-21481:
The failed ut not realted and they failed frequently in
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16781461#comment-16781461
]
Reid Chan commented on HBASE-21481:
---
Looks like some of tests are not very stable.
Shall i try again.
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16781362#comment-16781362
]
Hadoop QA commented on HBASE-21481:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16780580#comment-16780580
]
Hadoop QA commented on HBASE-21481:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16780285#comment-16780285
]
Guanghao Zhang commented on HBASE-21481:
+1 for 014 patch. Let's wait the HADOOP QA result.
>
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16780276#comment-16780276
]
Reid Chan commented on HBASE-21481:
---
There're some changes in tests to suit behavior {{skip the super
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16780271#comment-16780271
]
Hadoop QA commented on HBASE-21481:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16780140#comment-16780140
]
Hadoop QA commented on HBASE-21481:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16780092#comment-16780092
]
Reid Chan commented on HBASE-21481:
---
bq. For keep behavior compatible, maybe better to skip the super
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16780087#comment-16780087
]
Guanghao Zhang commented on HBASE-21481:
{quote}bq.So i want to keep it accord with other
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16780084#comment-16780084
]
Reid Chan commented on HBASE-21481:
---
User is told to configured with '@';
In acl znode, the '@' is
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16780078#comment-16780078
]
Guanghao Zhang commented on HBASE-21481:
{quote}Yes, you're right.
{quote}
So no need to keep
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16780062#comment-16780062
]
Reid Chan commented on HBASE-21481:
---
bq. After this, a super user can't revoke his permission, too?
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16780051#comment-16780051
]
Guanghao Zhang commented on HBASE-21481:
{code:java}
public static boolean isSuperUser(String
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16780048#comment-16780048
]
Guanghao Zhang commented on HBASE-21481:
v12 patch looks good. But there is one problem need to
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16780035#comment-16780035
]
Reid Chan commented on HBASE-21481:
---
I uploaded a wrong patch. (facepalm)
Reattached v12 patch.
>
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16780022#comment-16780022
]
Guanghao Zhang commented on HBASE-21481:
[~reidchan] The 012 patch is not related to this issue?
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16779350#comment-16779350
]
Hadoop QA commented on HBASE-21481:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16779209#comment-16779209
]
Reid Chan commented on HBASE-21481:
---
v12 removed super user information from {{AuthManager}}, and
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16778875#comment-16778875
]
Reid Chan commented on HBASE-21481:
---
Got your point, i will clean them up in v12.
> [acl] Superuser's
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16778857#comment-16778857
]
Guanghao Zhang commented on HBASE-21481:
Got it. The callee only is a String: user name or
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16778852#comment-16778852
]
Reid Chan commented on HBASE-21481:
---
Let's say: A grant|revoke B permissions, caller A(a User object)
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16778849#comment-16778849
]
Reid Chan commented on HBASE-21481:
---
Callee is not a User object, that's why we can't use that method.
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16778828#comment-16778828
]
Guanghao Zhang commented on HBASE-21481:
{quote}if (authManager.checkSuperPrivileges(name))
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16777838#comment-16777838
]
Reid Chan commented on HBASE-21481:
---
Failed tests passed on local with patch applied.
> [acl]
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=1616#comment-1616
]
Hadoop QA commented on HBASE-21481:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16777530#comment-16777530
]
Reid Chan commented on HBASE-21481:
---
Thanks for the review, [~zghaobac].
Addressed your comments in
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16776501#comment-16776501
]
Guanghao Zhang commented on HBASE-21481:
Sorry for late for this.
Why don't use
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16755847#comment-16755847
]
Hadoop QA commented on HBASE-21481:
---
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16755754#comment-16755754
]
Reid Chan commented on HBASE-21481:
---
ping [~zghaobac], It is planned to go in branch-2.2, as RM, would
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16755681#comment-16755681
]
Reid Chan commented on HBASE-21481:
---
v10 rebased master branch and fixed conflicts introduced by
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16745908#comment-16745908
]
Reid Chan commented on HBASE-21481:
---
ping [~busbey]
> [acl] Superuser's permissions should not be
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16709540#comment-16709540
]
Reid Chan commented on HBASE-21481:
---
Thank you, Sean!
> [acl] Superuser's permissions should not be
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16708790#comment-16708790
]
Sean Busbey commented on HBASE-21481:
-
shoot forgot to post here. I started reviewing this. will try
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16708248#comment-16708248
]
Hadoop QA commented on HBASE-21481:
---
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16708127#comment-16708127
]
Reid Chan commented on HBASE-21481:
---
Rebased master branch only, nothing changed in patch. And trigger
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16704187#comment-16704187
]
Reid Chan commented on HBASE-21481:
---
ping [~dbist13], [~psomogyi], WDYT.
> [acl] Superuser's
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16704185#comment-16704185
]
Reid Chan commented on HBASE-21481:
---
Failed test passed locally, not related i think.
{code}
[INFO]
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16703136#comment-16703136
]
Hadoop QA commented on HBASE-21481:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16702957#comment-16702957
]
Reid Chan commented on HBASE-21481:
---
v8's local test result:
{code}
[INFO]
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16702760#comment-16702760
]
Hadoop QA commented on HBASE-21481:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16702659#comment-16702659
]
Reid Chan commented on HBASE-21481:
---
Fixed TestRpcAccessChecks initialization problems.
Other failed
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16702099#comment-16702099
]
Hadoop QA commented on HBASE-21481:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16701747#comment-16701747
]
Reid Chan commented on HBASE-21481:
---
v6 addressed some of comments in RB.
> [acl] Superuser's
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16701327#comment-16701327
]
Reid Chan commented on HBASE-21481:
---
Yes, passed locally.
{{TestRpcAccessChecks.setup:141 »
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16701209#comment-16701209
]
Artem Ervits commented on HBASE-21481:
--
[~reidchan] your patch fails in
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16699883#comment-16699883
]
Reid Chan commented on HBASE-21481:
---
It's a non-trivial patch i think, need +1 from team member.
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16698908#comment-16698908
]
Hadoop QA commented on HBASE-21481:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16694902#comment-16694902
]
Hadoop QA commented on HBASE-21481:
---
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16694722#comment-16694722
]
Reid Chan commented on HBASE-21481:
---
ping [~stack], would you mind taking a look, if having free
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16694704#comment-16694704
]
Reid Chan commented on HBASE-21481:
---
v4 fixed checkstyle warning
> [acl] Superuser's permissions
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16694659#comment-16694659
]
Hadoop QA commented on HBASE-21481:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16690922#comment-16690922
]
Hadoop QA commented on HBASE-21481:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16690445#comment-16690445
]
Hadoop QA commented on HBASE-21481:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-21481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16690391#comment-16690391
]
Reid Chan commented on HBASE-21481:
---
* Add a check on target user/group where he is a superuser or a
60 matches
Mail list logo