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

Jason Lowe commented on TEZ-2549:
---------------------------------

Thanks for moving the patch forward, Jon.  Couple of comments:

- Do we need to do the same thing in Utils.java for COUNTER_GROUP_DISPLAY_NAME?
- Similar questions for DAGClientTimelineImpl.

A bit worried that if we sometimes expand the display name but not always then 
counter proto equals method could start returning weird results since it checks 
the displayname for equality.

> Reduce Counter Load on the Timeline Server
> ------------------------------------------
>
>                 Key: TEZ-2549
>                 URL: https://issues.apache.org/jira/browse/TEZ-2549
>             Project: Apache Tez
>          Issue Type: Sub-task
>            Reporter: Jonathan Eagles
>            Assignee: Jason Lowe
>         Attachments: TEZ-2549.1.branch-0.6.patch, TEZ-2549.1.patch, 
> ats-omit-dup-display-names-and-zero-counters_v2.patch
>
>




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

Reply via email to