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

kalyan kumar kalvagadda commented on SENTRY-1785:
-------------------------------------------------

There are couple of issues here
1. When Assume.assumeTrue fails the test is skipped but some times it is not 
skipped and the test fails.
2. setup(0 method in TestKrbConnectionTimeout calls setup again, which is 
reclusive call and there will be stack overflow.

Patch submitted has three changes
1. Updated the requires system property "sentry.hive.test.ticket.timeout" to 
true.
2. rename the BeforeClass annotated method of TestKrbConnectionTimeout class to 
avoid recursive call.
3. Invoked setup method to beginning of  BeforeClass annotated method of 
TestKrbConnectionTimeout.

> Fix TestKrbConnectionTimeout test
> ---------------------------------
>
>                 Key: SENTRY-1785
>                 URL: https://issues.apache.org/jira/browse/SENTRY-1785
>             Project: Sentry
>          Issue Type: Bug
>          Components: Sentry
>    Affects Versions: sentry-ha-redesign
>            Reporter: kalyan kumar kalvagadda
>            Assignee: kalyan kumar kalvagadda
>         Attachments: SENTRY-1785.001-sentry-ha-redesign.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to