You are viewing a plain text version of this content. The canonical link for it is here.

Posted to common-issues@hadoop.apache.org by GitBox <gi...@apache.org> on 2021/08/03 14:18:54 UTC

[GitHub] [hadoop] szilard-nemeth commented on pull request #3248: YARN-10874. Refactor NM ContainerLaunch#getEnvDependencies's unit tests

szilard-nemeth commented on pull request #3248:
URL: https://github.com/apache/hadoop/pull/3248#issuecomment-891886674


   Thanks @tomicooler for working on this.
   Nice patch, made the test way more easy to read.
   I think it's okay to add the jupiter dependency as it was already imported to hadoop-yarn-api project as well.
   +1, committed to trunk.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org