[REEF-949] Remove unnecessary TODO comment in Mesos-runtime module

• Type: Task
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 0.14
• Component/s: REEF-Runtime-Mesos
• Assignee: Dongjoon Hyun
• Reporter: Dongjoon Hyun
• Created: 15/Nov/15 16:32
• Updated: 16/Nov/15 00:36
• Resolved: 16/Nov/15 00:36

Description

At first look, I thought the following TODO comment means we need to add `MesosUser` parameter for `REEFScheduler`.

final Protos.FrameworkInfo frameworkInfo = Protos.FrameworkInfo.newBuilder()
        .setUser("") // TODO: make it configurable.

However, the current code already submits its executing user to Mesos correctly. I tested with `root` and `hadoop` account on Mesos 0.25.

In short, we had better remove the comment simply in order to prevent further confusion.

Issue Links

• links to: Link to PR