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

Jitendra Nath Pandey commented on HADOOP-7602:
----------------------------------------------

  I am ok with throwing exceptions in SecurityUtil methods instead of empty 
strings. One caveat is that these are public methods, changing signature could 
cause incompatibility. 
  I believed that canonical service name could be a more generic concept used 
in many contexts, therefore I have been reluctant to tie it down to only 
tokens. But as of now canonical service name is used only for token service 
currently, therefore returning null if token not supported should be ok. But 
again this is a public API, returning null is a semantic difference and may 
cause issues in applications already using this method and expecting a string 
back in all cases. 
  It may be unlikely that these methods are actually being used by any 
application, but these are part of public APIs so the compatibility point needs 
to be thought through.

> wordcount, sort etc on har files fails with NPE
> -----------------------------------------------
>
>                 Key: HADOOP-7602
>                 URL: https://issues.apache.org/jira/browse/HADOOP-7602
>             Project: Hadoop Common
>          Issue Type: Bug
>    Affects Versions: 0.20.205.0, 0.20.206.0
>            Reporter: John George
>            Assignee: John George
>             Fix For: 0.20.205.0
>
>         Attachments: hadoop-7602.1.patch, hadoop-7602.2.patch, 
> hadoop-7602.3.patch, hadoop-7602.4.patch, hadoop-7602.6.patch, 
> hadoop-7602.7.patch, hadoop-7602.8.patch, hadoop-7602.daryns_comment.2.patch, 
> hadoop-7602.daryns_comment.patch, hadoop-7602.patch
>
>
> wordcount, sort etc on har files fails with 
> NPE@createSocketAddr(NetUtils.java:137). 

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to