[ https://issues.apache.org/jira/browse/YARN-7246?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16191763#comment-16191763 ]
Eric Badger commented on YARN-7246: ----------------------------------- {noformat} +static char* USER_DOCKER_BINARY_PATH = "/bin/docker"; {noformat} A little nitpicky, but this var in the test should also be a const char*. Sorry for not seeing this on the previous review. Other than that, the patch looks good to me. > Fix the default docker binary path > ---------------------------------- > > Key: YARN-7246 > URL: https://issues.apache.org/jira/browse/YARN-7246 > Project: Hadoop YARN > Issue Type: Bug > Components: nodemanager > Reporter: Shane Kumpf > Assignee: Shane Kumpf > Priority: Blocker > Attachments: YARN-7246-branch-2.8.2.001.patch, > YARN-7246-branch-2.8.2.002.patch, YARN-7246-branch-2.8.2.003.patch, > YARN-7246-branch-2.8.2.004.patch > > -- This message was sent by Atlassian JIRA (v6.4.14#64029) --------------------------------------------------------------------- To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org