[
https://issues.apache.org/jira/browse/YARN-5656?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15507491#comment-15507491
]
Arun Suresh commented on YARN-5656:
---
+1 Committing this shortly..
> ReservationACLsTestBase fails on
[
https://issues.apache.org/jira/browse/YARN-5656?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15507475#comment-15507475
]
Sean Po commented on YARN-5656:
---
The Javadoc failures are not caused by the latest patch.
Changes were made
[
https://issues.apache.org/jira/browse/YARN-5656?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15505174#comment-15505174
]
Hadoop QA commented on YARN-5656:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-5656?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15497433#comment-15497433
]
Arun Suresh commented on YARN-5656:
---
Think you can remove the check from the testcase as well..
+1
[
https://issues.apache.org/jira/browse/YARN-5656?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15497412#comment-15497412
]
Sean Po commented on YARN-5656:
---
I think it is safe to remove the check entirely given the two scenarios:
1.
[
https://issues.apache.org/jira/browse/YARN-5656?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15496971#comment-15496971
]
Hadoop QA commented on YARN-5656:
-
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem ||
[
https://issues.apache.org/jira/browse/YARN-5656?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15496864#comment-15496864
]
Arun Suresh commented on YARN-5656:
---
Thanks for the patch [~seanpo03]..
Instead of removing the check