[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-10-04 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16191311#comment-16191311 ] Jason Lowe commented on YARN-7226: -- The unit test failures appear to be unrelated, and they pass for me

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

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

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

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

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-10-03 Thread Sidharta Seethana (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16190674#comment-16190674 ] Sidharta Seethana commented on YARN-7226: - [~jlowe] I committed the branch-2.8 patch.

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-10-03 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16190265#comment-16190265 ] Jason Lowe commented on YARN-7226: -- The findbugs warning is unrelated. > Whitelisted variables do not

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

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

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-10-03 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16190214#comment-16190214 ] Jason Lowe commented on YARN-7226: -- The unit test failure is interesting. It's a latent bug caused by

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

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

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

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

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-10-02 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16189200#comment-16189200 ] Hudson commented on YARN-7226: -- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #13009 (See

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-10-02 Thread Sidharta Seethana (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16189191#comment-16189191 ] Sidharta Seethana commented on YARN-7226: - [~jlowe] +1 to the latest patch - committed to trunk

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-10-02 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16188988#comment-16188988 ] Hadoop QA commented on YARN-7226: - | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-10-02 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16188981#comment-16188981 ] Hadoop QA commented on YARN-7226: - | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-10-02 Thread Sidharta Seethana (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16188820#comment-16188820 ] Sidharta Seethana commented on YARN-7226: - [~jlowe], recent changes to trunk (YARN-6550) seem to

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-09-29 Thread Sidharta Seethana (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16186627#comment-16186627 ] Sidharta Seethana commented on YARN-7226: - I'll take a look at the latest patch today or Monday at

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-09-28 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16184966#comment-16184966 ] Hadoop QA commented on YARN-7226: - | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-09-26 Thread Sidharta Seethana (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16181815#comment-16181815 ] Sidharta Seethana commented on YARN-7226: - [~jlowe], a couple of comments : * Is there a reason

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-09-26 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16181699#comment-16181699 ] Hadoop QA commented on YARN-7226: - | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-09-26 Thread Eric Badger (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16181019#comment-16181019 ] Eric Badger commented on YARN-7226: --- Hey [~jlowe], thanks for the patch. I have one question on it.

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-09-25 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16179941#comment-16179941 ] Hadoop QA commented on YARN-7226: - | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-09-25 Thread Sidharta Seethana (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16179698#comment-16179698 ] Sidharta Seethana commented on YARN-7226: - {quote} But how does this work today for Docker

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-09-25 Thread Eric Badger (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16179676#comment-16179676 ] Eric Badger commented on YARN-7226: --- [~jlowe], this approach makes sense to me. I don't see any case

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-09-25 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16179115#comment-16179115 ] Jason Lowe commented on YARN-7226: -- bq. If the user doesn't override the env vars, it is expected that

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-09-22 Thread Sidharta Seethana (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16177109#comment-16177109 ] Sidharta Seethana commented on YARN-7226: - [~ebadger], [~jlowe] {quote} The client gets to decide

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-09-22 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16176987#comment-16176987 ] Jason Lowe commented on YARN-7226: -- bq. So my vote would be for putting this patch in and then filing a

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-09-22 Thread Eric Badger (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16176961#comment-16176961 ] Eric Badger commented on YARN-7226: --- I can see the need for Docker containers to supply their own env

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-09-22 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16176376#comment-16176376 ] Jason Lowe commented on YARN-7226: -- bq. For example, an image used for running MR tasks could specify its

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-09-21 Thread Sidharta Seethana (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16175635#comment-16175635 ] Sidharta Seethana commented on YARN-7226: - {quote} As I understand it, a whitelisted variable is

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-09-21 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16174846#comment-16174846 ] Jason Lowe commented on YARN-7226: -- Pinging [~sidharta-s] and [~vvasudev] since the {{var=var:-value}}

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

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

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-09-20 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16173930#comment-16173930 ] Jason Lowe commented on YARN-7226: -- Test failures appear to be unrelated, but unfortunately the details

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

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

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

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

[jira] [Commented] (YARN-7226) Whitelisted variables do not support delayed variable expansion

2017-09-20 Thread Jason Lowe (JIRA)
[ https://issues.apache.org/jira/browse/YARN-7226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16173232#comment-16173232 ] Jason Lowe commented on YARN-7226: -- Maybe I'm missing something, but the whitelisted variable support