You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@sentry.apache.org by Anne Yu <an...@cloudera.com> on 2016/10/05 23:21:12 UTC

Review Request 52582: SENTRY-1489: Categorize e2e tests into slow and regular tests, so that can adapt the timeout and etc.

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

Review request for sentry, Hao Hao, Li Li, and Sravya Tirukkovalur.


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


Repository: sentry


Description
-------

org.apache.sentry.tests.e2e.dbprovider.TestDbCrossOperations and some other tests occasionally fail with time out exception. Should make a more flexible rule for different type of tests. By nature, some tests just take longer time to finish.


Diffs
-----

  sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/AbstractTestWithDbProvider.java e7cccbfb7955a497f485e3b69e9dc4d9db191473 
  sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbCrossOperations.java 60812cfa82d318ddfad32e439d3895f49ed21180 
  sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithHiveServer.java a314c0dde9c60e7dbb51156cea13980bf8600bf1 
  sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java 62da0256e8546a25f0ed0a97b8f3c21eec4fecaa 
  sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/RulesForE2ETest.java PRE-CREATION 
  sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/SlowE2ETest.java PRE-CREATION 

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


Testing
-------


Thanks,

Anne Yu