----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39928/ -----------------------------------------------------------
(Updated Nov. 6, 2015, 10:49 p.m.) Review request for sentry, Anne Yu, Lenni Kuff, and Sravya Tirukkovalur. Changes ------- Removed the exceptions in the test cases since it is no longer be thrown. Repository: sentry Description (updated) ------- Change-Id: I7c65bf182c44075f41de16943c5b7eb66e3dec0b SENTRY-994: Changed the Logger level and added more test case for removeFeature. Change-Id: I851344e088155e28c8978203759fe56884b29e41 SENTRY-994: Changes the use cases. Change-Id: I55a7a6c89c3ca42f28f85daa855caa9da5a86916 SENTRY-944: Removed the exception in the test cases. Change-Id: I5c33d5be357b4141f4e38f64f91b3f54058f5789 Diffs (updated) ----- sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryAuthorizationProvider.java 419ab68e0d03f995c55d229b762453468de47571 sentry-hdfs/sentry-hdfs-namenode-plugin/src/test/java/org/apache/sentry/hdfs/TestSentryAuthorizationProvider.java fd5146f079d93687738a522f42beaa59031a4f82 Diff: https://reviews.apache.org/r/39928/diff/ Testing ------- Added several new unit tests for setPermission/setUser/setGroup/removeAcl cases validation. Thanks, Hao Hao
