----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52582/#review154156 -----------------------------------------------------------
LGTM. Only one issue I am not sure: is it ok to resolve two jiras in the same patch? sentry-tests/sentry-tests-solr/pom.xml (line 83) <https://reviews.apache.org/r/52582/#comment223651> I am not sure if it is good to resolve two jiras in a same patch. Maybe better make this change in a separate patch? - Li Li On Oct. 24, 2016, 11:18 p.m., Anne Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52582/ > ----------------------------------------------------------- > > (Updated Oct. 24, 2016, 11:18 p.m.) > > > Review request for sentry, Alexander Kolbasov, 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 > sentry-tests/sentry-tests-solr/pom.xml > a60b4eed4dde6c182b061681305013b3bd548cc3 > > Diff: https://reviews.apache.org/r/52582/diff/ > > > Testing > ------- > > > Thanks, > > Anne Yu > >
