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/04/01 01:19:17 UTC

Re: Review Request 45502: SENTRY-1164: fix one e2e test failure on a real cluster; improved concurrent tests to be able to run locally.

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

(Updated March 31, 2016, 11:19 p.m.)


Review request for sentry and Hao Hao.


Changes
-------

Addressed Haohao's comments.


Bugs: sentry-1164
    https://issues.apache.org/jira/browse/sentry-1164


Repository: sentry


Description
-------

1. Fix one test failure in org.apache.sentry.tests.e2e.dbprovider.TestDbOperations.testCaseSensitivity;
2. Improve TestConcurrentClients to be able to run locally; also add more validation logics.


Diffs (updated)
-----

  sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestConcurrentClients.java bf871f01a8cdc0bab3e502c4fbdbee9285b8437b 
  sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java b96175797dfc8bf238722b6177a7800d569b7e8a 
  sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestOperations.java 7d3db2b135d3239410bb664dbf7aed5817ee01c0 

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


Testing
-------

Passed precommit build.


Thanks,

Anne Yu


Re: Review Request 45502: SENTRY-1164: fix one e2e test failure on a real cluster; improved concurrent tests to be able to run locally.

Posted by Hao Hao <ha...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45502/#review126602
-----------------------------------------------------------


Ship it!




Ship It!

- Hao Hao


On March 31, 2016, 11:19 p.m., Anne Yu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45502/
> -----------------------------------------------------------
> 
> (Updated March 31, 2016, 11:19 p.m.)
> 
> 
> Review request for sentry and Hao Hao.
> 
> 
> Bugs: sentry-1164
>     https://issues.apache.org/jira/browse/sentry-1164
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> 1. Fix one test failure in org.apache.sentry.tests.e2e.dbprovider.TestDbOperations.testCaseSensitivity;
> 2. Improve TestConcurrentClients to be able to run locally; also add more validation logics.
> 
> 
> Diffs
> -----
> 
>   sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestConcurrentClients.java bf871f01a8cdc0bab3e502c4fbdbee9285b8437b 
>   sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java b96175797dfc8bf238722b6177a7800d569b7e8a 
>   sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestOperations.java 7d3db2b135d3239410bb664dbf7aed5817ee01c0 
> 
> Diff: https://reviews.apache.org/r/45502/diff/
> 
> 
> Testing
> -------
> 
> Passed precommit build.
> 
> 
> Thanks,
> 
> Anne Yu
> 
>