Re: Review Request 40487: MESOS-3959: show slave hostname on executor page

2016-10-14 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40487/#review152729 --- Fix it, then Ship it! Looks like the file has been renamed.

Re: Review Request 40487: MESOS-3959: show slave hostname on executor page

2016-10-14 Thread Ian Babrou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40487/ --- (Updated Oct. 14, 2016, 8:19 p.m.) Review request for mesos and Benjamin

Re: Review Request 40487: MESOS-3959: show slave hostname on executor page

2016-07-11 Thread Benjamin Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40487/#review141773 --- Hey Ian, It does appear that we still do not have the hostname

Re: Review Request 40487: MESOS-3959: show slave hostname on executor page

2015-12-02 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40487/#review108774 --- Bad review! Reviews applied: [] Error: No reviewers specified.

Re: Review Request 40487: MESOS-3959: show slave hostname on executor page

2015-12-02 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40487/#review108765 --- Hey Ian thanks for your patch! While this is pretty trivial,