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

Billie Rinaldi commented on YARN-7777:
--------------------------------------

There is an additional problem, which is that RegistryDNS has its own matcher 
for username: 
https://github.com/apache/hadoop/blob/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-registry/src/main/java/org/apache/hadoop/registry/server/dns/BaseServiceRecordProcessor.java#L54

This matches the first part of the username containing word characters, 
[a-zA-Z_0-9]. So in the bob_dev case, patch 02 is converting the registry path 
to bob-dev, and RegistryDNS is converting that down to bob in the hostname. It 
seems like we should change the username matcher in BaseServiceRecordProcessor 
to match the requirements we are using for other elements of the hostname. We 
should probably also add into ServiceApiUtil a check that 
RegistryUtils.currentUser() matches the namePattern, instead of just checking 
its length like we are doing now.

> Fix user name format in YARN Registry DNS name 
> -----------------------------------------------
>
>                 Key: YARN-7777
>                 URL: https://issues.apache.org/jira/browse/YARN-7777
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Jian He
>            Assignee: Jian He
>            Priority: Major
>         Attachments: YARN-7777.01.patch, YARN-7777.02.patch
>
>
> user name that has "\_" should be converted to user "-", because DNS name 
> doesn't allow "_"



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to