[jira] [Updated] (SENTRY-583) Add boundary condition test coverage to HDFS synchronization test suite around max #of groups

[Anne Yu (JIRA)]

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

Anne Yu updated SENTRY-583:
---
Resolution: Fixed
Status: Resolved  (was: Patch Available)

commit d4165e423d57a230f8982c8dd795ac1d9dd48d0d
Author: Anne Yu 
Date:   Thu Apr 21 11:29:37 2016 -0700

SENTRY-583: Add boundary condition test coverage to HDFS synchronization 
test suite around max #of groups. (Anne Yu, reviewed by Haohao)

> Add boundary condition test coverage to HDFS synchronization test suite 
> around max #of groups
> -
>
> Key: SENTRY-583
> URL: https://issues.apache.org/jira/browse/SENTRY-583
> Project: Sentry
>  Issue Type: Test
>Affects Versions: 1.4.0, 1.5.0
>Reporter: Lenni Kuff
>Assignee: Anne Yu
> Fix For: 1.8.0
>
> Attachments: SENTRY-583-0.patch, SENTRY-583-1.patch, 
> SENTRY-583-2.patch, SENTRY-583-3.patch, SENTRY-583-4.patch, 
> SENTRY-583.5.patch, SENTRY-583.6.patch, SENTRY-583.7.patch, SENTRY-583.8.patch
>
>
> Normally, HDFS ACLs has a limit of 32 entries per object (HDFS-5617), but 
> this limit should not be enforced when using Sentry HDFS synchronization. I 
> verified that this works, but we should add a test case to cover this 
> scenario. Something like:
> # Grant to >32 unique groups to a table
> # Verify -getfacls returns all items
> It would also be interesting to see what happens when a new table is created 
> when a database has been granted >32 groups. The new table's directory will 
> inherit the permissions from the directory and I would assume that when the 
> feature is disabled some of these will be missing. It would be good to verify 
> that things don't blow up.



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

[jira] [Updated] (SENTRY-583) Add boundary condition test coverage to HDFS synchronization test suite around max #of groups

[Anne Yu (JIRA)]

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

Anne Yu updated SENTRY-583:
---
Attachment: SENTRY-583.8.patch

> Add boundary condition test coverage to HDFS synchronization test suite 
> around max #of groups
> -
>
> Key: SENTRY-583
> URL: https://issues.apache.org/jira/browse/SENTRY-583
> Project: Sentry
>  Issue Type: Test
>Affects Versions: 1.4.0, 1.5.0
>Reporter: Lenni Kuff
>Assignee: Anne Yu
> Fix For: 1.8.0
>
> Attachments: SENTRY-583-0.patch, SENTRY-583-1.patch, 
> SENTRY-583-2.patch, SENTRY-583-3.patch, SENTRY-583-4.patch, 
> SENTRY-583.5.patch, SENTRY-583.6.patch, SENTRY-583.7.patch, SENTRY-583.8.patch
>
>
> Normally, HDFS ACLs has a limit of 32 entries per object (HDFS-5617), but 
> this limit should not be enforced when using Sentry HDFS synchronization. I 
> verified that this works, but we should add a test case to cover this 
> scenario. Something like:
> # Grant to >32 unique groups to a table
> # Verify -getfacls returns all items
> It would also be interesting to see what happens when a new table is created 
> when a database has been granted >32 groups. The new table's directory will 
> inherit the permissions from the directory and I would assume that when the 
> feature is disabled some of these will be missing. It would be good to verify 
> that things don't blow up.



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

[jira] [Updated] (SENTRY-583) Add boundary condition test coverage to HDFS synchronization test suite around max #of groups

[Anne Yu (JIRA)]

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

Anne Yu updated SENTRY-583:
---
Attachment: (was: SENTRY-583.8.patch)

> Add boundary condition test coverage to HDFS synchronization test suite 
> around max #of groups
> -
>
> Key: SENTRY-583
> URL: https://issues.apache.org/jira/browse/SENTRY-583
> Project: Sentry
>  Issue Type: Test
>Affects Versions: 1.4.0, 1.5.0
>Reporter: Lenni Kuff
>Assignee: Anne Yu
> Fix For: 1.8.0
>
> Attachments: SENTRY-583-0.patch, SENTRY-583-1.patch, 
> SENTRY-583-2.patch, SENTRY-583-3.patch, SENTRY-583-4.patch, 
> SENTRY-583.5.patch, SENTRY-583.6.patch, SENTRY-583.7.patch
>
>
> Normally, HDFS ACLs has a limit of 32 entries per object (HDFS-5617), but 
> this limit should not be enforced when using Sentry HDFS synchronization. I 
> verified that this works, but we should add a test case to cover this 
> scenario. Something like:
> # Grant to >32 unique groups to a table
> # Verify -getfacls returns all items
> It would also be interesting to see what happens when a new table is created 
> when a database has been granted >32 groups. The new table's directory will 
> inherit the permissions from the directory and I would assume that when the 
> feature is disabled some of these will be missing. It would be good to verify 
> that things don't blow up.



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

[jira] [Updated] (SENTRY-583) Add boundary condition test coverage to HDFS synchronization test suite around max #of groups

[Anne Yu (JIRA)]

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

Anne Yu updated SENTRY-583:
---
Attachment: (was: SENTRY-583.8.patch)

> Add boundary condition test coverage to HDFS synchronization test suite 
> around max #of groups
> -
>
> Key: SENTRY-583
> URL: https://issues.apache.org/jira/browse/SENTRY-583
> Project: Sentry
>  Issue Type: Test
>Affects Versions: 1.4.0, 1.5.0
>Reporter: Lenni Kuff
>Assignee: Anne Yu
> Fix For: 1.8.0
>
> Attachments: SENTRY-583-0.patch, SENTRY-583-1.patch, 
> SENTRY-583-2.patch, SENTRY-583-3.patch, SENTRY-583-4.patch, 
> SENTRY-583.5.patch, SENTRY-583.6.patch, SENTRY-583.7.patch
>
>
> Normally, HDFS ACLs has a limit of 32 entries per object (HDFS-5617), but 
> this limit should not be enforced when using Sentry HDFS synchronization. I 
> verified that this works, but we should add a test case to cover this 
> scenario. Something like:
> # Grant to >32 unique groups to a table
> # Verify -getfacls returns all items
> It would also be interesting to see what happens when a new table is created 
> when a database has been granted >32 groups. The new table's directory will 
> inherit the permissions from the directory and I would assume that when the 
> feature is disabled some of these will be missing. It would be good to verify 
> that things don't blow up.



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

[jira] [Updated] (SENTRY-583) Add boundary condition test coverage to HDFS synchronization test suite around max #of groups

[Anne Yu (JIRA)]

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

Anne Yu updated SENTRY-583:
---
Attachment: SENTRY-583.8.patch

> Add boundary condition test coverage to HDFS synchronization test suite 
> around max #of groups
> -
>
> Key: SENTRY-583
> URL: https://issues.apache.org/jira/browse/SENTRY-583
> Project: Sentry
>  Issue Type: Test
>Affects Versions: 1.4.0, 1.5.0
>Reporter: Lenni Kuff
>Assignee: Anne Yu
> Fix For: 1.8.0
>
> Attachments: SENTRY-583-0.patch, SENTRY-583-1.patch, 
> SENTRY-583-2.patch, SENTRY-583-3.patch, SENTRY-583-4.patch, 
> SENTRY-583.5.patch, SENTRY-583.6.patch, SENTRY-583.7.patch, SENTRY-583.8.patch
>
>
> Normally, HDFS ACLs has a limit of 32 entries per object (HDFS-5617), but 
> this limit should not be enforced when using Sentry HDFS synchronization. I 
> verified that this works, but we should add a test case to cover this 
> scenario. Something like:
> # Grant to >32 unique groups to a table
> # Verify -getfacls returns all items
> It would also be interesting to see what happens when a new table is created 
> when a database has been granted >32 groups. The new table's directory will 
> inherit the permissions from the directory and I would assume that when the 
> feature is disabled some of these will be missing. It would be good to verify 
> that things don't blow up.



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

[jira] [Updated] (SENTRY-583) Add boundary condition test coverage to HDFS synchronization test suite around max #of groups

[Anne Yu (JIRA)]

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

Anne Yu updated SENTRY-583:
---
Attachment: SENTRY-583.7.patch

> Add boundary condition test coverage to HDFS synchronization test suite 
> around max #of groups
> -
>
> Key: SENTRY-583
> URL: https://issues.apache.org/jira/browse/SENTRY-583
> Project: Sentry
>  Issue Type: Test
>Affects Versions: 1.4.0, 1.5.0
>Reporter: Lenni Kuff
>Assignee: Anne Yu
> Fix For: 1.8.0
>
> Attachments: SENTRY-583-0.patch, SENTRY-583-1.patch, 
> SENTRY-583-2.patch, SENTRY-583-3.patch, SENTRY-583-4.patch, 
> SENTRY-583.5.patch, SENTRY-583.6.patch, SENTRY-583.7.patch
>
>
> Normally, HDFS ACLs has a limit of 32 entries per object (HDFS-5617), but 
> this limit should not be enforced when using Sentry HDFS synchronization. I 
> verified that this works, but we should add a test case to cover this 
> scenario. Something like:
> # Grant to >32 unique groups to a table
> # Verify -getfacls returns all items
> It would also be interesting to see what happens when a new table is created 
> when a database has been granted >32 groups. The new table's directory will 
> inherit the permissions from the directory and I would assume that when the 
> feature is disabled some of these will be missing. It would be good to verify 
> that things don't blow up.



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

[jira] [Updated] (SENTRY-583) Add boundary condition test coverage to HDFS synchronization test suite around max #of groups

[Anne Yu (JIRA)]

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

Anne Yu updated SENTRY-583:
---
Attachment: SENTRY-583.6.patch

[~haohao] I also posted a code review here, 
https://reviews.apache.org/r/46542/. Thanks for your help.

> Add boundary condition test coverage to HDFS synchronization test suite 
> around max #of groups
> -
>
> Key: SENTRY-583
> URL: https://issues.apache.org/jira/browse/SENTRY-583
> Project: Sentry
>  Issue Type: Test
>Affects Versions: 1.4.0, 1.5.0
>Reporter: Lenni Kuff
>Assignee: Anne Yu
> Fix For: 1.8.0
>
> Attachments: SENTRY-583-0.patch, SENTRY-583-1.patch, 
> SENTRY-583-2.patch, SENTRY-583-3.patch, SENTRY-583-4.patch, 
> SENTRY-583.5.patch, SENTRY-583.6.patch
>
>
> Normally, HDFS ACLs has a limit of 32 entries per object (HDFS-5617), but 
> this limit should not be enforced when using Sentry HDFS synchronization. I 
> verified that this works, but we should add a test case to cover this 
> scenario. Something like:
> # Grant to >32 unique groups to a table
> # Verify -getfacls returns all items
> It would also be interesting to see what happens when a new table is created 
> when a database has been granted >32 groups. The new table's directory will 
> inherit the permissions from the directory and I would assume that when the 
> feature is disabled some of these will be missing. It would be good to verify 
> that things don't blow up.



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