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

Hadoop QA commented on SENTRY-996:
----------------------------------

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

{color:red}Overall:{color} -1 due to an error

{color:red}ERROR:{color} mvn test exited 1

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

This message is automatically generated.

> Faulty loop logic in HdfsHAClientInvocationHandler + 
> HAClientInvocationHandler 
> -------------------------------------------------------------------------------
>
>                 Key: SENTRY-996
>                 URL: https://issues.apache.org/jira/browse/SENTRY-996
>             Project: Sentry
>          Issue Type: Bug
>            Reporter: Colm O hEigeartaigh
>            Assignee: Ke Jia
>         Attachments: SENTRY-996-002.patch, SENTRY-996.001.patch
>
>
> As discussed on the review board here: https://reviews.apache.org/r/41349/
> Both the HdfsHAClientInvocationHandler + HAClientInvocationHandler contain a 
> while(true) loop that looks like it was designed to catch certain exceptions 
> + retry. However, the logic is faulty in that it will always return after the 
> first iteration, not matter what (as picked up by PMD).



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

Reply via email to