[ https://issues.apache.org/jira/browse/SENTRY-996?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15301924#comment-15301924 ]
Hadoop QA commented on SENTRY-996: ---------------------------------- Here are the results of testing the latest attachment https://issues.apache.org/jira/secure/attachment/12806107/SENTRY-996.001.patch against master. {color:red}Overall:{color} -1 due to an error {color:red}ERROR:{color} failed to build with patch (exit code 1) Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/1639/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.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)