[jira] [Commented] (SENTRY-1155) Add waiting time for getMetastoreClient for avoiding metastore isn't ready

["Hadoop QA (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/SENTRY-1155?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15205989#comment-15205989 ] 

Hadoop QA commented on SENTRY-1155:
-----------------------------------

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12794709/SENTRY-1155.001.patch against master.

{color:red}Overall:{color} -1 due to 3 errors

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: org.apache.sentry.kafka.authorizer.SentryKafkaAuthorizerTest
{color:red}ERROR:{color} Failed: org.apache.sentry.kafka.authorizer.SentryKafkaAuthorizerTest

Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/1367/console

This message is automatically generated.

> Add waiting time for getMetastoreClient for avoiding metastore isn't ready
> --------------------------------------------------------------------------
>
>                 Key: SENTRY-1155
>                 URL: https://issues.apache.org/jira/browse/SENTRY-1155
>             Project: Sentry
>          Issue Type: Bug
>            Reporter: Dapeng Sun
>            Assignee: Dapeng Sun
>         Attachments: SENTRY-1155.001.patch
>
>
> For {{TestMetastoreEndToEnd}}, If metastore didn't finish the initialization.  context.getMetaStoreClient will throw exception. 
> {noformat}
> HiveMetaStoreClient client = context.getMetaStoreClient(ADMIN1);
> {noformat}
> We should wait for the initialization of metastore, for example:
> {noformat}
>     HiveMetaStoreClient client = null;
>     for (int i=0; i < 10; i++) {
>       try {
>         client = context.getMetaStoreClient(ADMIN1);
>         break;
>       } catch (Throwable e) {
>         // ignore
>       }
>       Thread.sleep(6000);
>     }
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)