[
https://issues.apache.org/jira/browse/YARN-9679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16908261#comment-16908261
]
Szilard Nemeth commented on YARN-9679:
--
Thanks [~adam.antal], resolved it!
> Regular code cleanup in
[
https://issues.apache.org/jira/browse/YARN-9679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16908259#comment-16908259
]
Adam Antal commented on YARN-9679:
--
Thank you for the commit [~snemeth]!
Half of the class looks
[
https://issues.apache.org/jira/browse/YARN-9679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16908209#comment-16908209
]
Hudson commented on YARN-9679:
--
FAILURE: Integrated in Jenkins build Hadoop-trunk-Commit #17130 (See
[
https://issues.apache.org/jira/browse/YARN-9679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16908198#comment-16908198
]
Szilard Nemeth commented on YARN-9679:
--
Hi [~adam.antal]!
As told on github, I merged your patch to
[
https://issues.apache.org/jira/browse/YARN-9679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16907462#comment-16907462
]
Szilard Nemeth commented on YARN-9679:
--
Hi [~adam.antal]!
Your PR now has conflicts against trunk.
[
https://issues.apache.org/jira/browse/YARN-9679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16902792#comment-16902792
]
Adam Antal commented on YARN-9679:
--
Got +1 from jenkins on the PR. Anyone fancy a review?
> Regular code
[
https://issues.apache.org/jira/browse/YARN-9679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16888085#comment-16888085
]
Adam Antal commented on YARN-9679:
--
Cleaned up the code according to your items except item 8: I argue