Github user zhitaoli commented on a diff in the pull request:
https://github.com/apache/mesos/pull/141#discussion_r71003361
--- Diff: include/mesos/mesos.proto ---
@@ -205,9 +205,12 @@ message MachineInfo {
* Describes a framework.
*/
message FrameworkInfo {
- // Used to determine the Unix user that an executor or task should
- // be launched as. If the user field is set to an empty string Mesos
- // will automagically set it to the current user.
+ // Used to determine the Unix user that an executor or task should be
+ // launched as. If the user field is set to an empty string
+ // MesosSchedulerDriver will automagically set it to the current user.
+ //
+ // NOTE: The user has to be set explicitly in case the HTTP API
+ // is used.
--- End diff --
Can we restructure these two sentences as this is not very clear.
Maybe consider something among:
```
// Used to determine the Unix user that an executor or task should be
// launched as.
//
// If using HTTP API, this field must be explicitly set.
//
// If using MesosSchedulerDriver and this field is set to an empty string,
MesosSchedulerDriver
// will automagically set it to the current user.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
