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

Brahma Reddy Battula commented on HDFS-8705:
--------------------------------------------

{{StringUtils.equalsIgnoreCase(..)}} tells equalsIgnoreCase is locale-free, I 
think, we no need to fix this jira...Had seen it used in so many places 
DFSAdmin.java,Balancer.java etc.. like below.
{code}
  if ("leave".equalsIgnoreCase(argv[idx])) {
      action = HdfsConstants.SafeModeAction.SAFEMODE_LEAVE;
{code}

[[email protected]] and [~walter.k.su] do you think so..? 

> BlockStoragePolicySuite uses equalsIgnoreCase for name lookup, won't work in 
> all locales
> ----------------------------------------------------------------------------------------
>
>                 Key: HDFS-8705
>                 URL: https://issues.apache.org/jira/browse/HDFS-8705
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: namenode
>    Affects Versions: 2.8.0
>            Reporter: Steve Loughran
>            Assignee: Brahma Reddy Battula
>            Priority: Minor
>         Attachments: HDFS-8705.patch
>
>
> Looking at {{BlockStoragePolicySuite.getPolicy(name)}}, is using 
> {{equalsIgnoreCase()}} to find a policy which matches a name.
> This will not work in all locales. It must use 
> {{toLowerCase(Locale.ENGLISH).equals(name)}}



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

Reply via email to