[jira] [Commented] (YARN-8734) Readiness check for remote service belonging to the same user

2018-09-28 Thread Eric Yang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16632535#comment-16632535 ] Eric Yang commented on YARN-8734: - Thank you [~billie.rinaldi]. > Readiness check for rem

[jira] [Commented] (YARN-8734) Readiness check for remote service belonging to the same user

2018-09-28 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16632135#comment-16632135 ] Hudson commented on YARN-8734: -- SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #150

[jira] [Commented] (YARN-8734) Readiness check for remote service belongs to the same user

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

[jira] [Commented] (YARN-8734) Readiness check for remote service belongs to the same user

2018-09-28 Thread Billie Rinaldi (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16631976#comment-16631976 ] Billie Rinaldi commented on YARN-8734: -- I kicked off a new build. > Readiness check

[jira] [Commented] (YARN-8734) Readiness check for remote service belongs to the same user

2018-09-28 Thread Billie Rinaldi (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16631972#comment-16631972 ] Billie Rinaldi commented on YARN-8734: -- Patch 6 looks fine to me. I tested this out w

[jira] [Commented] (YARN-8734) Readiness check for remote service belongs to the same user

2018-09-27 Thread Eric Yang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16631083#comment-16631083 ] Eric Yang commented on YARN-8734: - Precommit build failed due to other Jenkins problem. T

[jira] [Commented] (YARN-8734) Readiness check for remote service belongs to the same user

2018-09-26 Thread Eric Yang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16629545#comment-16629545 ] Eric Yang commented on YARN-8734: - [~gsaha] Good catch, I have corrected yml accordingly.

[jira] [Commented] (YARN-8734) Readiness check for remote service belongs to the same user

2018-09-26 Thread Eric Yang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16629464#comment-16629464 ] Eric Yang commented on YARN-8734: - [~gsaha], I see what you mean. It will be corrected i

[jira] [Commented] (YARN-8734) Readiness check for remote service belongs to the same user

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

[jira] [Commented] (YARN-8734) Readiness check for remote service belongs to the same user

2018-09-26 Thread Billie Rinaldi (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16629170#comment-16629170 ] Billie Rinaldi commented on YARN-8734: -- Oh, I see what you mean now, [~gsaha]. I didn

[jira] [Commented] (YARN-8734) Readiness check for remote service belongs to the same user

2018-09-26 Thread Gour Saha (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16629168#comment-16629168 ] Gour Saha commented on YARN-8734: - bq. Yes, component dependencies is also outside of comp

[jira] [Commented] (YARN-8734) Readiness check for remote service belongs to the same user

2018-09-24 Thread Billie Rinaldi (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16626598#comment-16626598 ] Billie Rinaldi commented on YARN-8734: -- Makes sense to me. Thanks, [~eyang]! > Readi

[jira] [Commented] (YARN-8734) Readiness check for remote service belongs to the same user

2018-09-24 Thread Eric Yang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16626565#comment-16626565 ] Eric Yang commented on YARN-8734: - [~billie.rinaldi] Good catch on the relationship betwee

[jira] [Commented] (YARN-8734) Readiness check for remote service

2018-09-22 Thread Billie Rinaldi (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16624780#comment-16624780 ] Billie Rinaldi commented on YARN-8734: -- I am fine with calling the field "dependencie

[jira] [Commented] (YARN-8734) Readiness check for remote service

2018-09-21 Thread Gour Saha (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16624346#comment-16624346 ] Gour Saha commented on YARN-8734: - Yup, it would be great to get [~billie.rinaldi] thought

[jira] [Commented] (YARN-8734) Readiness check for remote service

2018-09-21 Thread Eric Yang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16624340#comment-16624340 ] Eric Yang commented on YARN-8734: - {quote}In that case may be a simpler approach will be t

[jira] [Commented] (YARN-8734) Readiness check for remote service

2018-09-21 Thread Gour Saha (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16624319#comment-16624319 ] Gour Saha commented on YARN-8734: - In that case may be a simpler approach will be to call

[jira] [Commented] (YARN-8734) Readiness check for remote service

2018-09-21 Thread Eric Yang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16624306#comment-16624306 ] Eric Yang commented on YARN-8734: - [~gsaha] Local service would imply the current service

[jira] [Commented] (YARN-8734) Readiness check for remote service

2018-09-21 Thread Gour Saha (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16624250#comment-16624250 ] Gour Saha commented on YARN-8734: - [~eyang] this is a pretty useful feature so thanks for

[jira] [Commented] (YARN-8734) Readiness check for remote service

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

[jira] [Commented] (YARN-8734) Readiness check for remote service

2018-09-17 Thread Eric Yang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16618119#comment-16618119 ] Eric Yang commented on YARN-8734: - Patch 005 fixed the last checkstyle issue. > Readiness

[jira] [Commented] (YARN-8734) Readiness check for remote service

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

[jira] [Commented] (YARN-8734) Readiness check for remote service

2018-09-17 Thread Eric Yang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16617911#comment-16617911 ] Eric Yang commented on YARN-8734: - Patch 004 skips initialization of ServiceClient, if the

[jira] [Commented] (YARN-8734) Readiness check for remote service

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

[jira] [Commented] (YARN-8734) Readiness check for remote service

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

[jira] [Commented] (YARN-8734) Readiness check for remote service

2018-09-14 Thread Eric Yang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16615259#comment-16615259 ] Eric Yang commented on YARN-8734: - Patch 002 generalized the remote service dependency che

[jira] [Commented] (YARN-8734) Readiness check for remote service

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

[jira] [Commented] (YARN-8734) Readiness check for remote service

2018-09-13 Thread Eric Yang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16614199#comment-16614199 ] Eric Yang commented on YARN-8734: - Patch 001 is proof of concept to prevent service from d

[jira] [Commented] (YARN-8734) Readiness check for remote service

2018-09-10 Thread Eric Yang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16609797#comment-16609797 ] Eric Yang commented on YARN-8734: - Designed document is attached as "Dependency check vs.p

[jira] [Commented] (YARN-8734) Readiness check for remote service

2018-08-30 Thread Eric Yang (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16597902#comment-16597902 ] Eric Yang commented on YARN-8734: - [~billie.rinaldi] {quote}We might need to check the ind

[jira] [Commented] (YARN-8734) Readiness check for remote service

2018-08-30 Thread Billie Rinaldi (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16597711#comment-16597711 ] Billie Rinaldi commented on YARN-8734: -- Also, we should consider checking for cycles

[jira] [Commented] (YARN-8734) Readiness check for remote service

2018-08-30 Thread Billie Rinaldi (JIRA)
[ https://issues.apache.org/jira/browse/YARN-8734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16597710#comment-16597710 ] Billie Rinaldi commented on YARN-8734: -- We might need to check the individual compone