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

(Updated May 23, 2016, 8:56 p.m.)


Review request for sentry, Hao Hao, Lenni Kuff, and Sean Mackrory.


Changes
-------

Addressed code review comment by adding a test case without scheme.


Bugs: SENTRY-1230
    https://issues.apache.org/jira/browse/SENTRY-1230


Repository: sentry


Description
-------

SENTRY-1230: Add filesystem tests to test Sentry with user data storage on S3. 
The tests can also be run on other types of fileSystems (ms azure, hdfs). Just 
ensure pass correct parameters.
    For example,
       -Dsentry.e2etest.DFSType=S3DFS
       -Dsentry.e2etest.storage.uri=s3a://sentry-s3
       -Dfs.s3a.access.key=a
       -Dfs.s3a.secret.key=s
       export HIVE_CONF_DIR=/etc/hive/conf/hite-site.xml


Diffs (updated)
-----

  pom.xml a82c66d9a730c76403600b12db7563522d6c12be 
  sentry-tests/sentry-tests-hive/pom.xml 
e36b5c04d48b3211abaecd36bf2673e6304bdc31 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestDbHdfsBase.java
 001f5a49449112785cf129bfeda22c3bdf8cf3a6 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java
 0e4b3cab7685937e563b971274c66be43ba88fc1 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/fs/AbstractDFS.java
 32aabb41a1fef486ebed80571f0bd96e4c7c3c3c 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/fs/ClusterDFS.java
 06415cb828bfc6408bdc74d57c352ccfd55b7b43 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/fs/DFS.java
 67ba3386df6f52c1cd21d47eaec58a1a467bebcb 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/fs/DFSFactory.java
 7f650ceb01d6325c2dea794a207baaa5a4cb6e73 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/fs/S3DFS.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/fs/TestFSBase.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/fs/TestFSContants.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/fs/TestHiveWarehouseOnExtFs.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/fs/TestTableOnExtFS.java
 PRE-CREATION 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/AbstractHiveServer.java
 dda7dba662f7ba8fb9f705066c4bc680a7358f50 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/EmbeddedHiveServer.java
 52ba09e5e1006ff44a5073bce412ddee0774b24b 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServer.java
 175e84ceac8f24c1faa905d02daac1739e17f320 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/InternalHiveServer.java
 45f0ef28bc2393c16dd385cd8e37c133302b2993 
  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/UnmanagedHiveServer.java
 beae8e83bc9895f9c3aafd2eb9c6320b11ad0c1d 

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


Testing
-------

-------------------------------------------------------
 T E S T S
-------------------------------------------------------
Running org.apache.sentry.tests.e2e.hive.fs.TestHiveWarehouseOnExtFs
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 45.419 sec - in 
org.apache.sentry.tests.e2e.hive.fs.TestHiveWarehouseOnExtFs

Results :

Tests run: 1, Failures: 0, Errors: 0, Skipped: 0

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 1:28.585s
[INFO] Finished at: Fri May 20 17:09:38 PDT 2016
[INFO] Final Memory: 60M/825M
[INFO] ------------------------------------------------------------------------


Thanks,

Anne Yu

Reply via email to