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

Posted to issues@mesos.apache.org by "Joseph Wu (JIRA)" <ji...@apache.org> on 2015/11/09 23:23:11 UTC

[jira] [Commented] (MESOS-3864) Simplify and/or document the libprocess initialization synchronization logic

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

Joseph Wu commented on MESOS-3864:
----------------------------------

Reviews:
* Remove some stale comments: https://reviews.apache.org/r/39948/
* Document/simplify synchronization code: https://reviews.apache.org/r/39949/

> Simplify and/or document the libprocess initialization synchronization logic
> ----------------------------------------------------------------------------
>
>                 Key: MESOS-3864
>                 URL: https://issues.apache.org/jira/browse/MESOS-3864
>             Project: Mesos
>          Issue Type: Task
>          Components: libprocess
>            Reporter: Joseph Wu
>            Assignee: Joseph Wu
>            Priority: Minor
>              Labels: mesosphere
>
> Tracks this [TODO|https://github.com/apache/mesos/blob/3bda55da1d0b580a1b7de43babfdc0d30fbc87ea/3rdparty/libprocess/src/process.cpp#L749].
> The [synchronization logic of libprocess|https://github.com/apache/mesos/commit/cd757cf75637c92c438bf4cd22f21ba1b5be702f#diff-128d3b56fc8c9ec0176fdbadcfd11fc2] [predates abstractions|https://github.com/apache/mesos/commit/6c3b107e4e02d5ba0673eb3145d71ec9d256a639#diff-0eebc8689450916990abe080d86c2acb] like {{process::Once}}, which is used in almost all other one-time initialization blocks.  
> The logic should be documented.  It can also be simplified (see the [review description|https://reviews.apache.org/r/39949/]).  Or it can be replaced with {{process::Once}}.



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