You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sentry.apache.org by Alexander Kolbasov <ak...@gmail.com> on 2017/05/18 07:16:11 UTC

Review Request 59361: SENTRY-1772 Permissions created before table creation are not reflected in HDFS ACLs

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59361/
-----------------------------------------------------------

Review request for sentry, Brian Towles, Hao Hao, kalyan kumar kalvagadda, Na Li, Sergio Pena, and Vamsee Yarlagadda.


Bugs: SENTRY-1772
    https://issues.apache.org/jira/browse/SENTRY-1772


Repository: sentry


Description
-------

SENTRY-1772 Permissions created before table creation are not reflected in HDFS ACLs


Diffs
-----

  sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java 5dc0ad338bbfee37a537bdebbd3a24e76714b796 


Diff: https://reviews.apache.org/r/59361/diff/1/


Testing
-------


Thanks,

Alexander Kolbasov


Re: Review Request 59361: SENTRY-1772 Permissions created before table creation are not reflected in HDFS ACLs

Posted by Vamsee Yarlagadda <va...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59361/#review175468
-----------------------------------------------------------


Ship it!




Ship It!

- Vamsee Yarlagadda


On May 19, 2017, 4:46 a.m., Alexander Kolbasov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/59361/
> -----------------------------------------------------------
> 
> (Updated May 19, 2017, 4:46 a.m.)
> 
> 
> Review request for sentry, Brian Towles, Hao Hao, kalyan kumar kalvagadda, Na Li, Sergio Pena, and Vamsee Yarlagadda.
> 
> 
> Bugs: SENTRY-1772
>     https://issues.apache.org/jira/browse/SENTRY-1772
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> SENTRY-1772 Permissions created before table creation are not reflected in HDFS ACLs
> 
> 
> Diffs
> -----
> 
>   sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java 5dc0ad338bbfee37a537bdebbd3a24e76714b796 
>   sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/NotificationProcessor.java 083e0ac5a4a0a3243dba2ef95cf890101888ed59 
>   sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestHMSFollower.java fd97936ddcc34236a549b646786a9713bf4b4981 
> 
> 
> Diff: https://reviews.apache.org/r/59361/diff/2/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Alexander Kolbasov
> 
>


Re: Review Request 59361: SENTRY-1772 Permissions created before table creation are not reflected in HDFS ACLs

Posted by Alexander Kolbasov <ak...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59361/
-----------------------------------------------------------

(Updated May 19, 2017, 4:46 a.m.)


Review request for sentry, Brian Towles, Hao Hao, kalyan kumar kalvagadda, Na Li, Sergio Pena, and Vamsee Yarlagadda.


Changes
-------

Fixed HMSFollower unit test


Bugs: SENTRY-1772
    https://issues.apache.org/jira/browse/SENTRY-1772


Repository: sentry


Description
-------

SENTRY-1772 Permissions created before table creation are not reflected in HDFS ACLs


Diffs (updated)
-----

  sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java 5dc0ad338bbfee37a537bdebbd3a24e76714b796 
  sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/NotificationProcessor.java 083e0ac5a4a0a3243dba2ef95cf890101888ed59 
  sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestHMSFollower.java fd97936ddcc34236a549b646786a9713bf4b4981 


Diff: https://reviews.apache.org/r/59361/diff/2/

Changes: https://reviews.apache.org/r/59361/diff/1-2/


Testing
-------


Thanks,

Alexander Kolbasov