You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sentry.apache.org by Ryan Pridgeon <rn...@cloudera.com> on 2015/08/04 22:06:09 UTC

Re: Review Request 37086: DDL operations against view fail with Sentry Sync

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

(Updated Aug. 4, 2015, 8:06 p.m.)


Review request for sentry.


Changes
-------

More accurate summary


Summary (updated)
-----------------

DDL operations against view fail with Sentry Sync


Repository: sentry


Description
-------

SENTRY-780: HDFS Plugin should not execute path callbacks for views


Diffs
-----

  sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryMetastorePostEventListener.java 4924669 
  sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java d75c578 

Diff: https://reviews.apache.org/r/37086/diff/


Testing
-------

Create, Drop, Create or Replace as Views with Sentry-Sync enabled.


Thanks,

Ryan Pridgeon


Re: Review Request 37086: DDL operations against view fail with Sentry Sync

Posted by Ryan Pridgeon <rn...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37086/
-----------------------------------------------------------

(Updated Aug. 4, 2015, 9:25 p.m.)


Review request for sentry.


Changes
-------

+ new Diff


Repository: sentry


Description
-------

SENTRY-780: HDFS Plugin should not execute path callbacks for views


Diffs
-----

  sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryMetastorePostEventListener.java 4924669 
  sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java d75c578 

Diff: https://reviews.apache.org/r/37086/diff/


Testing
-------

Create, Drop, Create or Replace as Views with Sentry-Sync enabled.


File Attachments (updated)
----------------

SENTRY-780.patch.1
  https://reviews.apache.org/media/uploaded/files/2015/08/04/21f0e0ca-6757-4c60-bb18-7fdbfe076a6c__SENTRY-780.patch.1


Thanks,

Ryan Pridgeon