[ https://issues.apache.org/jira/browse/YARN-8209?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16459767#comment-16459767 ]
Jason Lowe commented on YARN-8209: ---------------------------------- The patch looks functional to me, so the following are all code cleanup comments. In the interest of unblocking 3.1.1 I'm OK with deferring these to another JIRA, but the non-nits should be addressed at some point. The instanceof checks are a bit hacky, and it would be nice to get those cleaned up. For example, DockerCommand could have a method that generates the PrivilegedOperation. The default implementation would generate the RUN_COMMAND operation with the command file, while DockerRmCommand and DockerInspectCommand can do their specializations without having the callers perform type checks to do the right thing. invoke_docker_command should not take a string of arguments that needs to be split but rather an arg array that is easier to pass to exec directly. Right now main.c has to build a string just so invoke_docker_command can take it apart. Nit: execvp is not going to return if it succeeds, so we can simplify the conditionals by removing the exit_code local and always returning DOCKER_RUN_FAILED. Nit: exec_docker_command may be a more appropriate name to indicate this is not expected to return. > NPE in DeletionService > ---------------------- > > Key: YARN-8209 > URL: https://issues.apache.org/jira/browse/YARN-8209 > Project: Hadoop YARN > Issue Type: Bug > Reporter: Chandni Singh > Assignee: Eric Badger > Priority: Major > Attachments: YARN-8209.001.patch, YARN-8209.002.patch > > > {code:java} > 2018-04-25 23:38:41,039 WARN concurrent.ExecutorHelper > (ExecutorHelper.java:logThrowableFromAfterExecute(63)) - Caught exception in > thread DeletionService #1: > java.lang.NullPointerException > at > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker.DockerClient.writeCommandToTempFile(DockerClient.java:109) > at > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker.DockerCommandExecutor.executeDockerCommand(DockerCommandExecutor.java:85) > at > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker.DockerCommandExecutor.executeStatusCommand(DockerCommandExecutor.java:192) > at > org.apache.hadoop.yarn.server.nodemanager.containermanager.linux.runtime.docker.DockerCommandExecutor.getContainerStatus(DockerCommandExecutor.java:128) > at > org.apache.hadoop.yarn.server.nodemanager.LinuxContainerExecutor.removeDockerContainer(LinuxContainerExecutor.java:935) > at > org.apache.hadoop.yarn.server.nodemanager.containermanager.deletion.task.DockerContainerDeletionTask.run(DockerContainerDeletionTask.java:61) > at > java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) > at java.util.concurrent.FutureTask.run(FutureTask.java:266) > at > java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:180) > at > java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:293) > at > java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) > at > java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) > at java.lang.Thread.run(Thread.java:748){code} -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org