[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15578769#comment-15578769 ] Hudson commented on YARN-5699: -- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10622 (See

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15578279#comment-15578279 ] Hadoop QA commented on YARN-5699: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15577056#comment-15577056 ] Rohith Sharma K S commented on YARN-5699: - Yes, current patch given based on trunk. Sure, I will

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15576895#comment-15576895 ] Sangjin Lee commented on YARN-5699: --- The latest patch LGTM. I'll wait until tomorrow to give folks time

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15575746#comment-15575746 ] Rohith Sharma K S commented on YARN-5699: - Test failures are unrelated to patch. YARN-5377 and

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15575722#comment-15575722 ] Hadoop QA commented on YARN-5699: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15575536#comment-15575536 ] Rohith Sharma K S commented on YARN-5699: - Updated the patch fixing review comment and checkstyle.

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15575534#comment-15575534 ] Rohith Sharma K S commented on YARN-5699: - Basically TestApplicationHistoryManagerOnTimelineStore

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15574347#comment-15574347 ] Sangjin Lee commented on YARN-5699: --- OK it's more of curiosity on my part as what this test does is

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15573867#comment-15573867 ] Rohith Sharma K S commented on YARN-5699: - bq. the name DIAGNOSTICS_INFO_INFO doesn't sound quite

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15573656#comment-15573656 ] Sangjin Lee commented on YARN-5699: --- Thanks for the update patch [~rohithsharma]! I think it's almost

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15572814#comment-15572814 ] Rohith Sharma K S commented on YARN-5699: - Test failure is not related to patch, i.e YARN-5679

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15571847#comment-15571847 ] Hadoop QA commented on YARN-5699: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15570899#comment-15570899 ] Sangjin Lee commented on YARN-5699: --- I'd like to know what would be made much easier with having the

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15570708#comment-15570708 ] Rohith Sharma K S commented on YARN-5699: - bq. So I'm not sure whether we need to store it again as

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15570387#comment-15570387 ] Li Lu commented on YARN-5699: - OK, I don't have a strong preference on that. Once the data is available on

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15570302#comment-15570302 ] Sangjin Lee commented on YARN-5699: --- The created time just seems redundant. Is there something that

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15570267#comment-15570267 ] Li Lu commented on YARN-5699: - Thanks [~sjlee0]. This said I'm generally fine with Rohith's planning. End time

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15570224#comment-15570224 ] Sangjin Lee commented on YARN-5699: --- Thanks for the clarification. Then what do you think of the

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15569980#comment-15569980 ] Li Lu commented on YARN-5699: - bq. Are you proposing elevating all event info items to the entity info unless

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15569971#comment-15569971 ] Sangjin Lee commented on YARN-5699: --- bq. What about end time? End time has to be get from event time

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15569771#comment-15569771 ] Li Lu commented on YARN-5699: - Actually let's take one step backward... What is the key benefit for keeping

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15568675#comment-15568675 ] Rohith Sharma K S commented on YARN-5699: - bq. Created time is at the topmost level in an entity.

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15568600#comment-15568600 ] Varun Saxena commented on YARN-5699: bq. It is only for easier accessibility rather than going through

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15567797#comment-15567797 ] Rohith Sharma K S commented on YARN-5699: - To me, goal for moving event info to entity info is

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15563620#comment-15563620 ] Li Lu commented on YARN-5699: - Thanks for the work [~rohithsharma]! Generally fine but a few concerns: 1. I

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15557347#comment-15557347 ] Varun Saxena commented on YARN-5699: bq. regarding the container state that was removed in YARN-5156;

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15556601#comment-15556601 ] Sangjin Lee commented on YARN-5699: --- Thanks [~rohithsharma] for the patch! Sorry it took me a while to

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15552342#comment-15552342 ] Varun Saxena commented on YARN-5699: For tracking URL part though, AHS will not be enabled with ATSv2

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15552332#comment-15552332 ] Varun Saxena commented on YARN-5699: Thanks [~rohithsharma] for the latest patch. Few comments: # In

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15552237#comment-15552237 ] Hadoop QA commented on YARN-5699: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15552187#comment-15552187 ] Rohith Sharma K S commented on YARN-5699: - Updated patch also fixes a another bug i.e always NM

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15552048#comment-15552048 ] Rohith Sharma K S commented on YARN-5699: - Updated patch by reverting changes made in

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551926#comment-15551926 ] Hadoop QA commented on YARN-5699: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551908#comment-15551908 ] Varun Saxena commented on YARN-5699: bq. Otherwise we miss the this information else container_state

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551812#comment-15551812 ] Naganarasimha G R commented on YARN-5699: - Had missed YARN-5156 let me check how to include that

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551803#comment-15551803 ] Naganarasimha G R commented on YARN-5699: - There was a detailed discussion on this and then we

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551786#comment-15551786 ] Rohith Sharma K S commented on YARN-5699: - bq. Is this intentional ? yes, because the same

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551754#comment-15551754 ] Varun Saxena commented on YARN-5699: [~rohithsharma], thanks for the patch. In the patch I see you have

[jira] [Commented] (YARN-5699) Retrospect yarn entity fields which are publishing in events info fields.

[ https://issues.apache.org/jira/browse/YARN-5699?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15551737#comment-15551737 ] Hadoop QA commented on YARN-5699: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||