[jira] [Commented] (AMBARI-2926) PropertyHelper.getPropertyCategory() does not account for replacement tokens in property id

["Tom Beerbower (JIRA)" <ji...@apache.org>]

    [ 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