----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51471/ -----------------------------------------------------------
(Updated Aug. 29, 2016, 6:41 p.m.) Review request for sentry, Hao Hao and Sravya Tirukkovalur. Changes ------- Addressed Haohao's comments. Bugs: SENTRY-1429 and SENTRY-1454 https://issues.apache.org/jira/browse/SENTRY-1429 https://issues.apache.org/jira/browse/SENTRY-1454 Repository: sentry Description ------- SENTRY-1429, SENTRY-1455: fix flaky TestHDFSIntegration, refactor this huge test class, improve its cleanUp and setUp for temp HDFS dirs; divide test class into smoke test class and p2 test class; in post commit run, we will only run smoke tests. Diffs (updated) ----- pom.xml b53e7766749c75f8e36097f624fd7a812c6d1761 sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java a2aead0c55cbee8aca9608539ad2681891acf57d sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java b0a3b6edc6201a9f902df1786f591b6bf3ac2250 sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationAdvanced.java PRE-CREATION sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationBase.java PRE-CREATION sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationEnd2End.java PRE-CREATION sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationWithHA.java be6d0822ea56509ae9171d2b79026b1c952c2ec9 Diff: https://reviews.apache.org/r/51471/diff/ Testing ------- Local. Thanks, Anne Yu