----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44558/#review123597 -----------------------------------------------------------
Ship it! Ship It! - Dapeng Sun On 三月 11, 2016, 2:39 p.m., Colin Ma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44558/ > ----------------------------------------------------------- > > (Updated 三月 11, 2016, 2:39 p.m.) > > > Review request for sentry. > > > Bugs: SENTRY-1123 > https://issues.apache.org/jira/browse/SENTRY-1123 > > > Repository: sentry > > > Description > ------- > > Delete the test cases in TestSqoopWildcardPrivilege, > TestSearchWildcardPrivilege, TestIndexerWildcardPrivilege, > TestDBWildcardPrivilege. Add test cases for Hive, Solr, Index, Sqoop with the > CommonPrivilege > > > Diffs > ----- > > > sentry-binding/sentry-binding-hive/src/test/java/org/apache/sentry/privilege/hive/TestCommonPrivilegeForHive.java > PRE-CREATION > > sentry-binding/sentry-binding-solr/src/test/java/org/apache/sentry/privilege/solr/TestCommonPrivilegeForSearch.java > PRE-CREATION > > sentry-binding/sentry-binding-sqoop/src/test/java/org/apache/sentry/privilege/sqoop/TestCommonPrivilegeForSqoop.java > PRE-CREATION > > sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/utils/SentryConstants.java > e752c76 > > sentry-policy/sentry-policy-common/src/main/java/org/apache/sentry/policy/common/CommonPrivilege.java > 8583448 > > sentry-policy/sentry-policy-db/src/test/java/org/apache/sentry/policy/db/TestDBWildcardPrivilege.java > 343a45d > > sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/TestCommonPrivilegeForIndexer.java > PRE-CREATION > > sentry-policy/sentry-policy-indexer/src/test/java/org/apache/sentry/policy/indexer/TestIndexerWildcardPrivilege.java > 3ecde2d > > sentry-policy/sentry-policy-search/src/test/java/org/apache/sentry/policy/search/TestSearchWildcardPrivilege.java > 688e3f8 > > sentry-policy/sentry-policy-sqoop/src/test/java/org/apache/sentry/policy/sqoop/TestSqoopWildcardPrivilege.java > 8d27726 > > Diff: https://reviews.apache.org/r/44558/diff/ > > > Testing > ------- > > > Thanks, > > Colin Ma > >
