Re: Review Request 68993: Introduced `logResponse` for http handlers.

2018-10-16 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68993/ --- (Updated Oct. 16, 2018, 11:57 a.m.) Review request for mesos, Benjamin

Re: Review Request 68993: Introduced `logResponse` for http handlers.

2018-10-16 Thread Till Toenshoff via Review Board
> On Oct. 11, 2018, 3:11 p.m., Benno Evers wrote: > > src/common/http.cpp > > Lines 1200 (patched) > > > > > > Since the outputs of this log line are likely to be used for > > rudimentary analysis using shell

Re: Review Request 68993: Introduced `logResponse` for http handlers.

2018-10-16 Thread Till Toenshoff via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68993/#review209609 --- Ship it! Modulo Benno's comment. src/common/http.hpp Lines

Re: Review Request 68993: Introduced `logResponse` for http handlers.

2018-10-11 Thread Benno Evers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68993/#review209455 --- Fix it, then Ship it! src/common/http.cpp Lines 1200

Review Request 68993: Introduced `logResponse` for http handlers.

2018-10-11 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68993/ --- Review request for mesos, Benjamin Hindman, Benno Evers, Benjamin Mahler,