[
https://issues.apache.org/jira/browse/MESOS-9040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16534291#comment-16534291
]
Till Toenshoff commented on MESOS-9040:
---------------------------------------
[~jamespeach] one reason on why it was likely not that popular in the past was
that several of our examples missed to implement the {{master}} flag
accordingly. I did at some point set that straight by
[73f6522|[https://github.com/apache/mesos/commit/73f6522905db5114d1bd6d99a3bab0793b14b86d].]
I still believe that this was a nice feature (hence my efforts) but I also see
how this renders a not so clean dependency tree. But, as you can see in my
commit, there is more done in the examples to make the entire feature perfectly
convenient - no need for manually setting up ACLs, derived roles and
authentication. It does indeed decrease friction a lot, especially for newbies.
I can see the following options;
# Kill that feature
# Have an additional (convenience) library for use in the examples that
provide this feature as Ben suggested
# Let the driver fork-exec towards `mesos-local` when signaled to do so
without reaping it
# Leave as is
I currently like (3) as it solves the linkage issue while still leaving the
feature intact.
Please note that the scheduler library also implements this feature, not just
the scheduler driver. See
https://github.com/apache/mesos/blob/16455857ee00e98147d8cb9fb6f31b22554dfe52/src/scheduler/scheduler.cpp#L191-L196
> Break scheduler driver dependency on mesos-local.
> -------------------------------------------------
>
> Key: MESOS-9040
> URL: https://issues.apache.org/jira/browse/MESOS-9040
> Project: Mesos
> Issue Type: Task
> Components: build, scheduler driver
> Reporter: James Peach
> Priority: Minor
>
> The scheduler driver in {{src/sched/sched.cpp}} has some special dependencies
> on the {{mesos-local}} code. This seems fairly hacky, but it also causes
> binary dependencies on {{src/local/local.cpp}} to be dragged into
> {{libmesos.so}}. {{libmesos.so}} would not otherwise require this code, which
> could be isolated in the {{mesos-local}} command.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)