[GitHub] [hbase] thangTang commented on issue #1510: HBASE-24184 listSnapshots returns empty when just use simple acl but not use authentication

2020-04-16 Thread GitBox
thangTang commented on issue #1510: HBASE-24184 listSnapshots returns empty when just use simple acl but not use authentication URL: https://github.com/apache/hbase/pull/1510#issuecomment-614545281 @infraio hi, could you help me to review this patch? ---

[GitHub] [hbase] thangTang commented on issue #1510: HBASE-24184 listSnapshots returns empty when just use simple acl but not use authentication

2020-04-16 Thread GitBox
thangTang commented on issue #1510: HBASE-24184 listSnapshots returns empty when just use simple acl but not use authentication URL: https://github.com/apache/hbase/pull/1510#issuecomment-614543028 @saintstack hi, could you help me to review this patch?

[GitHub] [hbase] thangTang commented on issue #1510: HBASE-24184 listSnapshots returns empty when just use simple acl but not use authentication

2020-04-14 Thread GitBox
thangTang commented on issue #1510: HBASE-24184 listSnapshots returns empty when just use simple acl but not use authentication URL: https://github.com/apache/hbase/pull/1510#issuecomment-613776774 The code in AccessController.preListSnapshot: `if (SnapshotDescriptionUtils.isSnapshotOwne

[GitHub] [hbase] thangTang commented on issue #1510: HBASE-24184 listSnapshots returns empty when just use simple acl but not use authentication

2020-04-14 Thread GitBox
thangTang commented on issue #1510: HBASE-24184 listSnapshots returns empty when just use simple acl but not use authentication URL: https://github.com/apache/hbase/pull/1510#issuecomment-613768352 The findbugs warning seems unrelated. --

[GitHub] [hbase] thangTang commented on issue #1510: HBASE-24184 listSnapshots returns empty when just use simple acl but not use authentication

2020-04-14 Thread GitBox
thangTang commented on issue #1510: HBASE-24184 listSnapshots returns empty when just use simple acl but not use authentication URL: https://github.com/apache/hbase/pull/1510#issuecomment-613768194 @enis Hi, could you help me to review this patch? -