----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22240/ -----------------------------------------------------------
(Updated June 4, 2014, 7:35 p.m.) Review request for sentry, Jarek Cecho, Prasad Mujumdar, and Sravya Tirukkovalur. Changes ------- Updating the Diff : Please review TODO note in TestDbSandboxOps (line 38) Repository: sentry Description ------- Couple of Test case fixes to get the filter push down patch to work. NOTE : I have placed some TODOs where I make a few assumptions.. kindly review Diffs (updated) ----- sentry-policy/sentry-policy-db/src/main/java/org/apache/sentry/policy/db/DBModelAuthorizables.java f4b32e1 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/SimpleDBProviderBackend.java 326b91d sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java ff656aa sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbEndToEnd.java 9c0c8b5 sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbSandboxOps.java 5eef792 sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegeAtTransform.java 732632b sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestSandboxOps.java 10c7b82 Diff: https://reviews.apache.org/r/22240/diff/ Testing ------- Thanks, Arun Suresh
