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

Naganarasimha G R commented on MAPREDUCE-6688:
----------------------------------------------

Hi [~varun_saxena], 
Earlier when we had discussed the solution offline i did not get the complete 
picture but after putting more thought on it and seeing your patch, had my 
reservations on the approach which i mentioned 
{{HistoryEvent.getTimelineEntitiesToPublish}}, In my view overall design across 
the history events seems to have specific modifications which are required for 
JobFinished and JobSubmitted Event. Also if we plan to push later through RM 
then i am not sure we will try to push events from EventHandler to RM. And also 
configuration would be difficult to understand to the user as it considers the 
implementation specific logic.

Even if we take [~gtCarrera9]'s approach {{HistoryEvent.addDataToEntity}} will 
also have its own set of issues.
So, IMO if its important to capture configuration as part of System Entities 
and if its important split the configs then i am fine with the current approach 
provided if others are ok to refactor it again based on the approach taken in 
YARN-3401.

> Store job configurations in Timeline Service v2
> -----------------------------------------------
>
>                 Key: MAPREDUCE-6688
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-6688
>             Project: Hadoop Map/Reduce
>          Issue Type: Sub-task
>          Components: applicationmaster
>    Affects Versions: YARN-2928
>            Reporter: Junping Du
>            Assignee: Varun Saxena
>              Labels: yarn-2928-1st-milestone
>         Attachments: MAPREDUCE-6688-YARN-2928.01.patch, 
> YARN-3959-YARN-2928.01.patch
>
>
> We already have configuration field in HBase schema for application entity. 
> We need to make sure AM write it out when it get launched.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org

Reply via email to