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

Tsuyoshi Ozawa commented on MAPREDUCE-5905:
-------------------------------------------

+1, committing this shortly. This patch fixes a mistake about the management of 
array indices.

Before the fix:
{quote}
 
{(group1)(group1)[(counter1)(counter1)(45)][(counter2)(counter2)(45)][(counter\{\}\(\)\[\])(counter\{\}\(\)\[\])(45)]}{(group2)(group2)[(counter1)(counter1)(45)][(counter2)(counter2)(45)][(counter\{\}\(\)\[\])(counter\{\}\(\)\[\])(45)]}{(group\{\}\(\)\[\])(group\{\}\(\)\[\])[(counter1)(counter1)(45)][(counter2)(counter2)(45)][(counter\{\}\(\)\[\])(counter\{\}\(\)\[\])(45)]}nullnullnullnullnullnull
{quote}

After the fix:
{quote}
 
{(group1)(group1)[(counter1)(counter1)(45)][(counter2)(counter2)(45)][(counter\{\}\(\)\[\])(counter\{\}\(\)\[\])(45)]}{(group2)(group2)[(counter1)(counter1)(45)][(counter2)(counter2)(45)][(counter\{\}\(\)\[\])(counter\{\}\(\)\[\])(45)]}{(group\{\}\(\)\[\])(group\{\}\(\)\[\])[(counter1)(counter1)(45)][(counter2)(counter2)(45)][(counter\{\}\(\)\[\])(counter\{\}\(\)\[\])(45)]}
{quote}

> CountersStrings.toEscapedCompactStrings outputs unnecessary "null" strings
> --------------------------------------------------------------------------
>
>                 Key: MAPREDUCE-5905
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-5905
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>    Affects Versions: 2.4.0, 2.7.0
>            Reporter: Akira AJISAKA
>            Assignee: Akira AJISAKA
>            Priority: Minor
>         Attachments: MAPREDUCE-5905.patch
>
>
> CountersStrings.toEscapedCompactStrings outputs "null" strings if a 
> CounterGroup has more than one Counter.
> That way there are some "null" strings in MRv1(CDH) job history log.
> https://issues.cloudera.org/browse/DISTRO-598



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

Reply via email to