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

Review request for sentry.


Repository: sentry


Description
-------

1) Created a new maven module sentry-tests/sentry-test-hive for the hive e2e 
tests
2) Moved all the sentry hive e2e tests from the java package: 
org.apache.sentry.tests.e2e.* to org.apache.sentry.tests.e2e.hive.*

NOTE: I don't know why in reviewboard it's showing up as the files being 
deleted and recreated.  In gitg for example I see:
"rename from sentry-tests/src/test/resources/sentry-site.xml
rename to sentry-tests/sentry-tests-hive/src/test/resources/sentry-site.xml"

The only changes to those files besides being renamed are the imports change 
due to 2) above


Diffs
-----

  sentry-tests/pom.xml 5536348 
  sentry-tests/sentry-tests-hive/pom.xml PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithHiveServer.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticDFS.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticLocalFS.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/Context.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/DummySentryOnFailureHook.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/PolicyFileEditor.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestCrossDbOps.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestEndToEnd.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestExportImportPrivileges.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestMetadataObjectRetrieval.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestMetadataPermissions.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestMovingToProduction.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPerDBConfiguration.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPerDatabasePolicyFile.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegeAtTransform.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtDatabaseScope.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtFunctionScope.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtTableScope.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestRuntimeMetadataRetrieval.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestSandboxOps.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestSentryOnFailureHookLoading.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestServerConfiguration.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestUriPermissions.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestUserManagement.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/AbstractHiveServer.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/EmbeddedHiveServer.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/ExternalHiveServer.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServer.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServerFactory.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/InternalHiveServer.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/UnmanagedHiveServer.java
 PRE-CREATION 
  sentry-tests/sentry-tests-hive/src/test/resources/access-site.xml 
PRE-CREATION 
  sentry-tests/sentry-tests-hive/src/test/resources/emp.dat PRE-CREATION 
  sentry-tests/sentry-tests-hive/src/test/resources/hive-site.xml PRE-CREATION 
  sentry-tests/sentry-tests-hive/src/test/resources/kv1.dat PRE-CREATION 
  sentry-tests/sentry-tests-hive/src/test/resources/log4j.properties 
PRE-CREATION 
  sentry-tests/sentry-tests-hive/src/test/resources/sentry-site.xml 
PRE-CREATION 
  sentry-tests/sentry-tests-hive/src/test/resources/test-authz-provider.ini 
PRE-CREATION 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/AbstractTestWithHiveServer.java
 d4060d3 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/AbstractTestWithStaticConfiguration.java
 40232c8 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/AbstractTestWithStaticDFS.java
 ab30fa5 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/AbstractTestWithStaticLocalFS.java
 c29c16c 
  sentry-tests/src/test/java/org/apache/sentry/tests/e2e/Context.java 30d1aaa 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/DummySentryOnFailureHook.java
 87d1b01 
  sentry-tests/src/test/java/org/apache/sentry/tests/e2e/PolicyFileEditor.java 
69bf2ad 
  sentry-tests/src/test/java/org/apache/sentry/tests/e2e/TestCrossDbOps.java 
c822863 
  sentry-tests/src/test/java/org/apache/sentry/tests/e2e/TestEndToEnd.java 
c45dfbc 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/TestExportImportPrivileges.java
 22fe430 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/TestMetadataObjectRetrieval.java
 24684f2 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/TestMetadataPermissions.java
 f3d493f 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/TestMovingToProduction.java
 c7b5e31 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/TestPerDBConfiguration.java
 8d520fc 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/TestPerDatabasePolicyFile.java
 8c0fb71 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/TestPrivilegeAtTransform.java
 53f295d 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/TestPrivilegesAtDatabaseScope.java
 69bfddc 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/TestPrivilegesAtFunctionScope.java
 25746c1 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/TestPrivilegesAtTableScope.java
 ed4509e 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/TestRuntimeMetadataRetrieval.java
 b9f71a9 
  sentry-tests/src/test/java/org/apache/sentry/tests/e2e/TestSandboxOps.java 
70c460e 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/TestSentryOnFailureHookLoading.java
 43469c5 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/TestServerConfiguration.java
 2c7ed7e 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/TestUriPermissions.java 
51ced04 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/TestUserManagement.java 
ed9fe36 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/hiveserver/AbstractHiveServer.java
 0c2fbc3 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/hiveserver/EmbeddedHiveServer.java
 48c112c 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/hiveserver/ExternalHiveServer.java
 6065002 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/hiveserver/HiveServer.java
 ce4b3e8 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/hiveserver/HiveServerFactory.java
 af3344e 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/hiveserver/InternalHiveServer.java
 7e0fed7 
  
sentry-tests/src/test/java/org/apache/sentry/tests/e2e/hiveserver/UnmanagedHiveServer.java
 288d2f7 
  sentry-tests/src/test/resources/access-site.xml e4de05a 
  sentry-tests/src/test/resources/emp.dat 5922b20 
  sentry-tests/src/test/resources/hive-site.xml 237c408 
  sentry-tests/src/test/resources/kv1.dat 20fb0dc 
  sentry-tests/src/test/resources/log4j.properties d941816 
  sentry-tests/src/test/resources/sentry-site.xml de0c9cf 
  sentry-tests/src/test/resources/test-authz-provider.ini 014d827 

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


Testing
-------

Ran "mvn test -Pdownload-hadoop" successfully


Thanks,

Gregory Chanan

Reply via email to