[
https://issues.apache.org/jira/browse/YARN-5210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15369762#comment-15369762
]
Hudson commented on YARN-5210:
--
SUCCESS: Integrated in Hadoop-trunk-Commit #10074 (See
[
https://issues.apache.org/jira/browse/YARN-5210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15323859#comment-15323859
]
Varun Saxena commented on YARN-5210:
Thanks [~vrushalic] for the review and commit.
Thanks
[
https://issues.apache.org/jira/browse/YARN-5210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15322931#comment-15322931
]
Vrushali C commented on YARN-5210:
--
+ 1 to merge (as discussed on the ATS call).
> NPE in Distributed
[
https://issues.apache.org/jira/browse/YARN-5210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15321955#comment-15321955
]
Joep Rottinghuis commented on YARN-5210:
Yeah, we need to also discuss whether YARN-5170 should go
[
https://issues.apache.org/jira/browse/YARN-5210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15321845#comment-15321845
]
Varun Saxena commented on YARN-5210:
We still have some JIRAs' to close before we drop onto trunk.
[
https://issues.apache.org/jira/browse/YARN-5210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15321744#comment-15321744
]
Li Lu commented on YARN-5210:
-
Hmm, just realized we're closing the branch for now. I'll make sure this commit
[
https://issues.apache.org/jira/browse/YARN-5210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15321314#comment-15321314
]
Li Lu commented on YARN-5210:
-
Thanks [~varun_saxena]! Patch LGTM, will commit later today if no other
[
https://issues.apache.org/jira/browse/YARN-5210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15319435#comment-15319435
]
Varun Saxena commented on YARN-5210:
Yes. In TestDistributedShell, we do not check for specific events.
[
https://issues.apache.org/jira/browse/YARN-5210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15319277#comment-15319277
]
Li Lu commented on YARN-5210:
-
LGTM. Did we miss this in TestDistributedShell all the time?
> NPE in
[
https://issues.apache.org/jira/browse/YARN-5210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15319189#comment-15319189
]
Varun Saxena commented on YARN-5210:
QA report comes out clean.
Kindly review.
Haven't added tests
[
https://issues.apache.org/jira/browse/YARN-5210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15319176#comment-15319176
]
Hadoop QA commented on YARN-5210:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-5210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15319104#comment-15319104
]
Varun Saxena commented on YARN-5210:
Tested and verified the fixes made in patch above.
As can be seen
[
https://issues.apache.org/jira/browse/YARN-5210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15319096#comment-15319096
]
Varun Saxena commented on YARN-5210:
I have submitted a patch to fix the issues.
Marked it for 1st
13 matches
Mail list logo