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

(Updated 三月 22, 2016, 6:55 p.m.)


Review request for sentry and Colin Ma.


Changes
-------

Update patch according comments


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


Repository: sentry


Description
-------

For TestMetastoreEndToEnd, If metastore didn't finish the initialization. 
context.getMetaStoreClient will throw exception.
    HiveMetaStoreClient client = context.getMetaStoreClient(ADMIN1);
We should wait for the initialization of metastore, for example:
    HiveMetaStoreClient client = null;
    for (int i=0; i < 10; i++) {
      try {
        client = context.getMetaStoreClient(ADMIN1);
        break;
      } catch (Throwable e) {
        // ignore
      }
      Thread.sleep(6000);
    }


Diffs (updated)
-----

  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/Context.java
 2e508d6 

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


Testing
-------


Thanks,

Dapeng Sun

Reply via email to