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

(Updated Oct. 28, 2014, 9:55 p.m.)


Review request for sentry.


Changes
-------

Feedback based on review comments


Repository: sentry


Description
-------

SENTRY-507 adding two more items to banned config vals in getConfigVal()


Diffs (updated)
-----

  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyStoreProcessor.java
 6de9992 
  
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceIntegration.java
 23bd765 

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


Testing
-------

Unit testing; added new tests for additional items.


Thanks,

Mike Yoder

Reply via email to