[jira] [Commented] (SENTRY-1797) SentryKerberosContext should use periodic executor instead of managing periodic execution via run() method.

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

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

Hadoop QA commented on SENTRY-1797:
-----------------------------------

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12952082/sentry-1797.1.patch against master.

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

{color:red}ERROR:{color} mvn test exited 1
{color:red}ERROR:{color} Failed: org.apache.sentry.provider.db.log.appender.TestRollingFileWithoutDeleteAppender

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

This message is automatically generated.

> SentryKerberosContext should use periodic executor instead of managing periodic execution via run() method.
> -----------------------------------------------------------------------------------------------------------
>
>                 Key: SENTRY-1797
>                 URL: https://issues.apache.org/jira/browse/SENTRY-1797
>             Project: Sentry
>          Issue Type: Bug
>          Components: Sentry
>    Affects Versions: 1.8.0, 2.0.0
>            Reporter: Alexander Kolbasov
>            Assignee: Brian Towles
>            Priority: Major
>              Labels: bite-sized, newbie
>         Attachments: sentry-1797.1.patch
>
>
> The SentryKerberosContext class tries to simulate periodic executor using while loop with a sleep in the run() method. Instead it should use periodic executor.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)