----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51353/ -----------------------------------------------------------
(Updated Aug. 24, 2016, 5:07 p.m.) Review request for sentry and Hao Hao. Changes ------- Added some comments to address previous review. Bugs: SENTRY-1447 https://issues.apache.org/jira/browse/SENTRY-1447 Repository: sentry Description ------- We start to see more e2e test failures on s3 clusters. For example, org.apache.sentry.tests.e2e.hdfs.TestDbHdfsExtMaxGroups need to be ignored for Sentry on S3; If run org.apache.sentry.tests.e2e.hive.fs.TestHiveWarehouseOnExtFs.TestCreateTable, have to correctly configure "-Dsentry.e2etest.storage.uri=s3a://sentry-s3"; Add exec(statement, "set hive.exim.uri.scheme.whitelist=hdfs,pfile,s3a;") for import/export tests. Also fixed on long test class causes time out failure. Diffs (updated) ----- sentry-tests/sentry-tests-hive/pom.xml 2979b45810b04665c2c66275a0b8544b5f2065d8 sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestColumnEndToEnd.java 8c152576d452b3069258e7f95c93cbf42e34e325 sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbCrossOperations.java PRE-CREATION sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestDbHdfsBase.java e545c37c182f3d77995323659ae539a8ad1de0cb sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestExportImportPrivileges.java b38eaaf35ec97978fef06fe91e3e5f5e391d4cc3 sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestOperationsPart2.java 7186f1af8159c30064e5ae3b5b1900d4556e1e4a Diff: https://reviews.apache.org/r/51353/diff/ Testing ------- On a test cluster. Thanks, Anne Yu