----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31018/#review72976 -----------------------------------------------------------
sentry-policy/sentry-policy-search/src/test/java/org/apache/sentry/policy/search/TestSearchModelAuthorizables.java <https://reviews.apache.org/r/31018/#comment119102> Is this an intentional change? I mean looks like this is related to Search and we included it in the lily hbase support jira. Do we want to create a separate patch for this? Or is it okay to commit as part of this patch? - Vamsee Yarlagadda On Feb. 13, 2015, 7:32 p.m., Gregory Chanan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31018/ > ----------------------------------------------------------- > > (Updated Feb. 13, 2015, 7:32 p.m.) > > > Review request for sentry, shen guoquan and Vamsee Yarlagadda. > > > Repository: sentry > > > Description > ------- > > Fairly straightforward translation from sentry-policy-search. > > One naming fix in sentry-policy-search. > > > Diffs > ----- > > pom.xml 8905862023f2ad0ab160d022f33c7a4e04e7fd3a > sentry-dist/pom.xml 67c6641a8c005b73ab93f34b8f681af327efc1e7 > sentry-policy/pom.xml dfd3d8f35bdf2389f84352de4b119d6c8ffbf453 > sentry-policy/sentry-policy-indexer/pom.xml PRE-CREATION > > sentry-policy/sentry-policy-indexer/src/main/java/org/apache/sentry/policy/indexer/AbstractIndexerPrivilegeValidator.java > PRE-CREATION > > sentry-policy/sentry-policy-indexer/src/main/java/org/apache/sentry/policy/indexer/IndexerModelAuthorizables.java > PRE-CREATION > > sentry-policy/sentry-policy-indexer/src/main/java/org/apache/sentry/policy/indexer/IndexerRequiredInPrivilege.java > PRE-CREATION > > sentry-policy/sentry-policy-indexer/src/main/java/org/apache/sentry/policy/indexer/IndexerWildcardPrivilege.java > PRE-CREATION > > sentry-policy/sentry-policy-indexer/src/main/java/org/apache/sentry/policy/indexer/SimpleIndexerPolicyEngine.java > PRE-CREATION > > sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/AbstractTestIndexerPolicyEngine.java > PRE-CREATION > > sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/IndexerPolicyFileBackend.java > PRE-CREATION > > sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/TestIndexerAuthorizationProviderGeneralCases.java > PRE-CREATION > > sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/TestIndexerAuthorizationProviderSpecialCases.java > PRE-CREATION > > sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/TestIndexerModelAuthorizables.java > PRE-CREATION > > sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/TestIndexerPolicyEngineDFS.java > PRE-CREATION > > sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/TestIndexerPolicyEngineLocalFS.java > PRE-CREATION > > sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/TestIndexerPolicyNegative.java > PRE-CREATION > > sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/TestIndexerRequiredInRole.java > PRE-CREATION > > sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/TestIndexerWildcardPrivilege.java > PRE-CREATION > sentry-policy/sentry-policy-indexer/src/test/resources/log4j.properties > PRE-CREATION > > sentry-policy/sentry-policy-indexer/src/test/resources/test-authz-provider.ini > PRE-CREATION > > sentry-policy/sentry-policy-search/src/test/java/org/apache/sentry/policy/search/TestSearchModelAuthorizables.java > c2511cdaf259cd0617bfc581e24a92e7623b28d2 > > Diff: https://reviews.apache.org/r/31018/diff/ > > > Testing > ------- > > Ran the unit tests. > > > Thanks, > > Gregory Chanan > >
