[ 
https://issues.apache.org/jira/browse/HADOOP-6938?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12905985#action_12905985
 ] 

Kan Zhang commented on HADOOP-6938:
-----------------------------------

ran ant test and test-patch locally. All passed. Javadoc warnings are 
unrelated. Also ran previously-failed TestMRCLI tests with the patch, they 
passed as well.

     [exec] -1 overall.  
     [exec] 
     [exec]     +1 @author.  The patch does not contain any @author tags.
     [exec] 
     [exec]     +1 tests included.  The patch appears to include 3 new or 
modified tests.
     [exec] 
     [exec]     -1 javadoc.  The javadoc tool appears to have generated 1 
warning messages.
     [exec] 
     [exec]     +1 javac.  The applied patch does not increase the total number 
of javac compiler warnings.
     [exec] 
     [exec]     +1 findbugs.  The patch does not introduce any new Findbugs 
warnings.
     [exec] 
     [exec]     +1 release audit.  The applied patch does not increase the 
total number of release audit warnings.
     [exec] 
     [exec]     +1 system tests framework.  The patch passed system tests 
framework compile.


> ConnectionId.getRemotePrincipal() should check if security is enabled
> ---------------------------------------------------------------------
>
>                 Key: HADOOP-6938
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6938
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: ipc, security
>            Reporter: Kan Zhang
>            Assignee: Kan Zhang
>         Attachments: c6938-01.patch, c6938-03.patch, c6938-04.patch
>
>
> When security is not enabled, getRemotePrincipal() should return null, which 
> means the Kerberos principal of the remote server is ignored. This bug was 
> caught by TestCLI on Yahoo 20S branch.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to