[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-06-01 Thread Daryn Sharp (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16033233#comment-16033233 ] Daryn Sharp commented on YARN-6245: --- Posted to new YARN-6679 in case you wish to pursue the immutable

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-06-01 Thread Daryn Sharp (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16033054#comment-16033054 ] Daryn Sharp commented on YARN-6245: --- I've been OOO. I'll be posting my collection of patches today for

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-05-22 Thread Arun Suresh (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16020266#comment-16020266 ] Arun Suresh commented on YARN-6245: --- [~roniburd], is this similar to what you had proposed in YARN-6418 ?

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-05-22 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16020182#comment-16020182 ] Wangda Tan commented on YARN-6245: -- [~daryn], Discussed with [~jlowe] offline, it looks like a great

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-05-22 Thread Daryn Sharp (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16019701#comment-16019701 ] Daryn Sharp commented on YARN-6245: --- [~jlowe] asked me to comment since we're running into 2.8 scheduler

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-04-12 Thread Roni Burd (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15966166#comment-15966166 ] Roni Burd commented on YARN-6245: - Sounds good guys. I'm having some issues with the change in trunk (I was

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-04-04 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15955811#comment-15955811 ] Wangda Tan commented on YARN-6245: -- [~roniburd], for patch submission, I agree with [~kasha] to submit one

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-04-04 Thread Karthik Kambatla (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15955780#comment-15955780 ] Karthik Kambatla commented on YARN-6245: bq. It works very well so I think we may not need a huge

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-04-04 Thread Roni Burd (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15955593#comment-15955593 ] Roni Burd commented on YARN-6245: - I agree. The scheduler should use only LightResource only. In addition

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-04-03 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15954489#comment-15954489 ] Wangda Tan commented on YARN-6245: -- yeah, so basically if there's any interacts to other processes like

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-04-03 Thread Karthik Kambatla (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15954477#comment-15954477 ] Karthik Kambatla commented on YARN-6245: One other thing. The scheduler should probably only use

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-04-03 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15954320#comment-15954320 ] Wangda Tan commented on YARN-6245: -- Sounds good [~kasha]. > Add FinalResource object to reduce overhead

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-04-03 Thread Karthik Kambatla (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15954300#comment-15954300 ] Karthik Kambatla commented on YARN-6245: We are in agreement. {{LightResource}} seems very useful

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-04-03 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15954295#comment-15954295 ] Wangda Tan commented on YARN-6245: -- [~kasha], actually I'm not sure if we need the

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-04-03 Thread Karthik Kambatla (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15954232#comment-15954232 ] Karthik Kambatla commented on YARN-6245: I think I understand some more now. :) Now, I see how the

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-04-03 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15954180#comment-15954180 ] Wangda Tan commented on YARN-6245: -- bq. I see the difference between LightResource and FinalResource and

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-04-03 Thread Karthik Kambatla (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15954014#comment-15954014 ] Karthik Kambatla commented on YARN-6245: [~leftnoteasy] - I see the difference between

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-04-03 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15953885#comment-15953885 ] Wangda Tan commented on YARN-6245: -- [~roniburd], All your suggestions make sense to me. In addition to

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-04-01 Thread Roni Burd (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15952107#comment-15952107 ] Roni Burd commented on YARN-6245: - We have run SLS for a couple of weeks, and we observed that when we have

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-03-31 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15951449#comment-15951449 ] Wangda Tan commented on YARN-6245: -- [~kasha], Sorry for the long delay of response, I missed your

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-03-01 Thread Karthik Kambatla (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15890378#comment-15890378 ] Karthik Kambatla commented on YARN-6245: [~leftnoteasy] - I didn't understand you question about

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-02-27 Thread Wangda Tan (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15886660#comment-15886660 ] Wangda Tan commented on YARN-6245: -- Thanks [~kasha], bq. What do you think of an approach where

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-02-27 Thread Karthik Kambatla (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15886640#comment-15886640 ] Karthik Kambatla commented on YARN-6245: Barely skimmed through the patch. What do you think of

[jira] [Commented] (YARN-6245) Add FinalResource object to reduce overhead of Resource class instancing

2017-02-27 Thread Karthik Kambatla (JIRA)
[ https://issues.apache.org/jira/browse/YARN-6245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15886632#comment-15886632 ] Karthik Kambatla commented on YARN-6245: I am very much in favor of doing this; was even