Review Request 45153: SENTRY-1155: Add waiting time for getMetastoreClient for avoiding metastore isn't ready

[Dapeng Sun]

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

Review request for sentry and Colin Ma.


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
-----

  
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/metastore/TestMetastoreEndToEnd.java
 98ec814 

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


Testing
-------


Thanks,

Dapeng Sun