Re: Review Request 43267: Returned "ServiceUnavailable" for slave's /state during recovery.

2016-02-08 Thread Vinod Kone
> On Feb. 6, 2016, 2:26 p.m., Klaus Ma wrote: > > src/slave/http.cpp, line 402 > > > > > > For GUI developer, it's better to export more info when it's > > recovering, e.g. build_time, build_date. So I'd like to

Re: Review Request 43267: Returned "ServiceUnavailable" for slave's /state during recovery.

2016-02-08 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43267/ --- (Updated Feb. 9, 2016, 12:40 a.m.) Review request for mesos, Anand Mazumdar

Re: Review Request 43267: Returned "ServiceUnavailable" for slave's /state during recovery.

2016-02-08 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43267/#review118339 --- Ship it! LGTM - Anand Mazumdar On Feb. 9, 2016, 12:40 a.m.,

Re: Review Request 43267: Returned "ServiceUnavailable" for slave's /state during recovery.

2016-02-07 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43267/#review118193 --- Looks good. Just a couple of minor issues.

Re: Review Request 43267: Returned "ServiceUnavailable" for slave's /state during recovery.

2016-02-06 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43267/#review118159 --- Patch looks great! Reviews applied: [43267] Passed command:

Re: Review Request 43267: Returned "ServiceUnavailable" for slave's /state during recovery.

2016-02-06 Thread Klaus Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43267/#review118143 --- src/slave/http.cpp (line 402)

Review Request 43267: Returned "ServiceUnavailable" for slave's /state during recovery.

2016-02-05 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43267/ --- Review request for mesos, Anand Mazumdar and Ben Mahler. Bugs: MESOS-4066