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

Junping Du commented on YARN-3276:
----------------------------------

Fix title to reflect what we discussion above. 
Thanks [~zjshen] for review and comments! Will update a patch to fix this.
In addition, I think we cannot add static method to existing TimelineUtils 
directly because it is inside hadoop-yarn-common project (module) which depends 
on hadoop-yarn-api (or it will cause bi-directional dependency in maven). 
However, I think we can refactor the existing TimelineUtils to hadoop-yarn-api 
then add this method. Thoughts? 

> Refactor and fix null casting in some map cast for TimelineEntity (old and 
> new)
> -------------------------------------------------------------------------------
>
>                 Key: YARN-3276
>                 URL: https://issues.apache.org/jira/browse/YARN-3276
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: timelineserver
>            Reporter: Junping Du
>            Assignee: Junping Du
>         Attachments: YARN-3276-v2.patch, YARN-3276.patch
>
>
> Per discussion in YARN-3087, we need to refactor some similar logic to cast 
> map to hashmap and get rid of NPE issue.



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

Reply via email to