[jira] [Commented] (MESOS-6767) Reached unreachable statement at /mesos/src/slave/containerizer/mesos/launch.cpp:766

["Kevin Klues (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/MESOS-6767?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15743166#comment-15743166 ] 

Kevin Klues commented on MESOS-6767:
------------------------------------

{noformat}
commit 739e52803e937cd98ef97768ac656ac2059cc722
Author: Jie Yu <yu...@gmail.com>
Date:   Mon Dec 12 11:20:23 2016 -0800

    Fixed an unreachable statement in launch.cpp.
    
    The statement is actually reachable if exec fails. This patch fix it
    so that it is actually unreachable.
    
    Review: https://reviews.apache.org/r/54673
{noformat}

> Reached unreachable statement at <path>/mesos/src/slave/containerizer/mesos/launch.cpp:766
> ------------------------------------------------------------------------------------------
>
>                 Key: MESOS-6767
>                 URL: https://issues.apache.org/jira/browse/MESOS-6767
>             Project: Mesos
>          Issue Type: Bug
>            Reporter: Kevin Klues
>            Assignee: Jie Yu
>            Priority: Blocker
>              Labels: containerizer, mesosphere
>
> This error message can pop up in unexpected places (e.g. when running a LAUNCH_NESTED_CONTAINER_SESSION and an invalid command is passed to it).
> We should likely just remove the UNREACHABLE() statement here as it's obviously reachable in cases where the command we are trying to launch is not found.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)