[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16455443#comment-16455443
]
Hudson commented on YARN-7939:
--
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #14070 (See
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16454799#comment-16454799
]
Chandni Singh commented on YARN-7939:
-
Thanks [~eyang] for reviewing, testing, and merging the patch.
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16454786#comment-16454786
]
Hudson commented on YARN-7939:
--
SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #14067 (See
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16454270#comment-16454270
]
Shane Kumpf commented on YARN-7939:
---
Thanks for the patch, [~csingh]! Overall, this looks good to me. I
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16453303#comment-16453303
]
genericqa commented on YARN-7939:
-
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16453289#comment-16453289
]
Chandni Singh commented on YARN-7939:
-
Please note {{hadoop.yarn.client.api.impl.TestAMRMProxy}} passes
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16453182#comment-16453182
]
Chandni Singh commented on YARN-7939:
-
[~eyang] Patch 12 has just checkstyle fix which got introduced
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16453161#comment-16453161
]
genericqa commented on YARN-7939:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16452928#comment-16452928
]
Chandni Singh commented on YARN-7939:
-
[~eyang] Uploaded patch 11. I have tested this against LCE.
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16452543#comment-16452543
]
Eric Yang commented on YARN-7939:
-
{quote}I don't understand what you mean by this. We throw a message that
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16451518#comment-16451518
]
Chandni Singh commented on YARN-7939:
-
[~eyang] you are correct. There is existing AM transition logic
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16451435#comment-16451435
]
Eric Yang commented on YARN-7939:
-
[~csingh] Thank you for patch 10. It resolved the Invalid event:
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16451434#comment-16451434
]
genericqa commented on YARN-7939:
-
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16451279#comment-16451279
]
Chandni Singh commented on YARN-7939:
-
[~eyang] patch 10 handles the fixable checkstyle warnings and
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16450448#comment-16450448
]
Eric Yang commented on YARN-7939:
-
[~csingh] Thank you for the patch. 12 of the 39 checkstyle issues can
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16449430#comment-16449430
]
genericqa commented on YARN-7939:
-
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16449328#comment-16449328
]
Chandni Singh commented on YARN-7939:
-
[~eyang] Uploaded patch 9 that handles the following:
1. Manage
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16448982#comment-16448982
]
Chandni Singh commented on YARN-7939:
-
Couple of bugs discovered by [~shaneku...@gmail.com]
1. If the
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16448549#comment-16448549
]
Eric Yang commented on YARN-7939:
-
[~csingh] [~shaneku...@gmail.com] Upgrade log attached for your review.
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16446513#comment-16446513
]
Chandni Singh commented on YARN-7939:
-
Created https://issues.apache.org/jira/browse/YARN-8194 to
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16446382#comment-16446382
]
Eric Yang commented on YARN-7939:
-
By reverting YARN-7973, the error messages disappeared, and I see the
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16446132#comment-16446132
]
Eric Yang commented on YARN-7939:
-
[~csingh] AM log is attached as serviceam.log.
> Yarn Service Upgrade:
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16446122#comment-16446122
]
Chandni Singh commented on YARN-7939:
-
[~eyang] can you please attach the AM log?
> Yarn Service
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16446061#comment-16446061
]
Eric Yang commented on YARN-7939:
-
[~csingh] This is not a docker app. There is nothing related to docker
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16446053#comment-16446053
]
Chandni Singh commented on YARN-7939:
-
[~eyang] Yes I am using the latest code. I have YARN-7973
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16446042#comment-16446042
]
Eric Yang commented on YARN-7939:
-
[~csingh] Thank you for the patch. I am getting this error message in
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16445261#comment-16445261
]
genericqa commented on YARN-7939:
-
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16445117#comment-16445117
]
Chandni Singh commented on YARN-7939:
-
Patch 8 contains the fix that aligns the state of the component
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16437773#comment-16437773
]
Chandni Singh commented on YARN-7939:
-
[~eyang]
{quote}
2018-04-13 17:23:08,321 ERROR
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16437696#comment-16437696
]
Eric Yang commented on YARN-7939:
-
[~csingh] With tarball application, patch 007 shows this diagnostics
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16436950#comment-16436950
]
genericqa commented on YARN-7939:
-
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16436827#comment-16436827
]
genericqa commented on YARN-7939:
-
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16436616#comment-16436616
]
Chandni Singh commented on YARN-7939:
-
[~eyang] The container may have failed which is why you see this
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16436497#comment-16436497
]
Eric Yang commented on YARN-7939:
-
[~csingh] I am getting the following exception in AM:
{code}
2018-04-12
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16434839#comment-16434839
]
genericqa commented on YARN-7939:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16434781#comment-16434781
]
genericqa commented on YARN-7939:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16431423#comment-16431423
]
genericqa commented on YARN-7939:
-
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16431149#comment-16431149
]
Gour Saha commented on YARN-7939:
-
bq. There is no {{NEEDS_UPGRADE}} state at service level, so the json
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16430951#comment-16430951
]
Chandni Singh commented on YARN-7939:
-
[~eyang]
Upgraded is either 2 steps when finalization is done
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16424494#comment-16424494
]
Eric Yang commented on YARN-7939:
-
If the service is in flexing mode, the first request NEEDS_UPGRADE state
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16424424#comment-16424424
]
Eric Yang commented on YARN-7939:
-
Base on patch 3 and design document, the upgrade logic is performed as:
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16419780#comment-16419780
]
Gour Saha commented on YARN-7939:
-
[~eyang], for all modifications to a service in a service collection,
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16418343#comment-16418343
]
genericqa commented on YARN-7939:
-
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16418315#comment-16418315
]
genericqa commented on YARN-7939:
-
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16418198#comment-16418198
]
Eric Yang commented on YARN-7939:
-
[~gsaha] {quote}
What do you mean by this?
{quote}
Nothing in
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16418184#comment-16418184
]
Chandni Singh commented on YARN-7939:
-
[~eyang] [~gsaha] I found few problems while testing the patch.
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16418079#comment-16418079
]
Gour Saha commented on YARN-7939:
-
bq. Sorry, this part was messed up during the original implementation.
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16418070#comment-16418070
]
Eric Yang commented on YARN-7939:
-
[~csingh] It would be nice if we clean up user submitted state to:
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16418014#comment-16418014
]
Chandni Singh commented on YARN-7939:
-
[~gsaha]
{quote}Yes, we should use UPGRADING instead of UPGRADE
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16417988#comment-16417988
]
Gour Saha commented on YARN-7939:
-
Yes, we should use UPGRADING instead of UPGRADE (which is an action
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16417969#comment-16417969
]
Chandni Singh commented on YARN-7939:
-
[~eyang] thanks for reviewing the patch.
I will add the CLI
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16417916#comment-16417916
]
Eric Yang commented on YARN-7939:
-
[~csingh] When user submit a curl PUT, the ServiceState needs to be
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16417907#comment-16417907
]
Eric Yang commented on YARN-7939:
-
[~csingh] Thank you for the patch. At high level, the patch looks good.
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16416537#comment-16416537
]
genericqa commented on YARN-7939:
-
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16416479#comment-16416479
]
genericqa commented on YARN-7939:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-7939?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16416469#comment-16416469
]
genericqa commented on YARN-7939:
-
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
56 matches
Mail list logo
