> On June 25, 2013, 10:23 p.m., Ben Mahler wrote: > > src/webui/master/static/controllers.js, line 214 > > <https://reviews.apache.org/r/11814/diff/3/?file=304657#file304657line214> > > > > I'm ok with this as a hack for now. Can you update the comment: > > > > // If the executor ID is empty, this is a command executor with an > > internal executor ID generated from the task ID. > > // TODO(brenden): Remove this once > > https://issues.apache.org/jira/browse/MESOS-527 is fixed.
Okay, I'll update this stuff in a bit. I've got one more Web UI patch coming after this one as well. I was out for the last 5 days so I'm catching up still. - Brenden ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11814/#review22376 ----------------------------------------------------------- On June 13, 2013, 4:42 p.m., Brenden Matthews wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11814/ > ----------------------------------------------------------- > > (Updated June 13, 2013, 4:42 p.m.) > > > Review request for mesos. > > > Repository: mesos > > > Description > ------- > > Fix executor IDs in web UI. > > Some frameworks don't report an executor ID, but have a task ID. As a > workaround, we can use the task ID when the executor ID is empty. > > Review: https://reviews.apache.org/r/11814 > > > Diffs > ----- > > src/webui/master/static/controllers.js > 0ea77cec2d6ca520e69fd74eb56fe05d32b711bd > src/webui/master/static/framework.html > c2d770bc40fae90ed6e3bb6b4b37cad551d69956 > > Diff: https://reviews.apache.org/r/11814/diff/ > > > Testing > ------- > > Tested by test driving the web UI. > > > Thanks, > > Brenden Matthews > >
