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

Tom Beerbower commented on AMBARI-2926:
---------------------------------------

Proposal :

In PropertyHelper.getPropertyCategory(), check to see if the given property id 
contains replacement tokens. If it does then look for the case where a / is 
part of a replacement token and ignore it as a delimiter.

                
> PropertyHelper.getPropertyCategory() does not account for replacement tokens 
> in property id
> -------------------------------------------------------------------------------------------
>
>                 Key: AMBARI-2926
>                 URL: https://issues.apache.org/jira/browse/AMBARI-2926
>             Project: Ambari
>          Issue Type: Bug
>            Reporter: Tom Beerbower
>            Assignee: Tom Beerbower
>
> The utility method PropertyHelper.getPropertyCategory(propertyId) does not 
> account for replacement tokens in property ids.  These were recently added to 
> allow for dynamic metric names.  For example ...
> {code}
> metrics/yarn/Queue/$1.replaceAll(",q(\d+)=","/").substring(1)
> {code}
> Since the / character is used as a delimiter, the / in the replacement token 
> method argument will cause the category for the above property id to be 
> reported as ...
> {code}
> metrics/yarn/Queue/$1.replaceAll(",q(\d+)=","
> {code}
> ... instead of ...
> {code}
> metrics/yarn/Queue
> {code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to