Re: Review Request 61849: Improved consistency of cout/cerr and glog usage in main functions.

2017-09-01 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61849/#review184345 --- Ship it! Ship It! - Till Toenshoff On Aug. 31, 2017, 4:39

Re: Review Request 61849: Improved consistency of cout/cerr and glog usage in main functions.

2017-08-31 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61849/#review184327 --- Patch looks great! Reviews applied: [61849] Passed command:

Re: Review Request 61849: Improved consistency of cout/cerr and glog usage in main functions.

2017-08-31 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61849/#review184306 --- Bad patch! Reviews applied: [61849] Logs available here:

Re: Review Request 61849: Improved consistency of cout/cerr and glog usage in main functions.

2017-08-31 Thread Armand Grillet
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61849/ --- (Updated Aug. 31, 2017, 4:39 p.m.) Review request for mesos, Andrei Budnik and

Re: Review Request 61849: Improved consistency of cout/cerr and glog usage in main functions.

2017-08-31 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61849/#review184269 --- Fix it, then Ship it! src/launcher/default_executor.cpp Line

Re: Review Request 61849: Improved consistency of cout/cerr and glog usage in main functions.

2017-08-29 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61849/#review184077 --- Patch looks great! Reviews applied: [61849] Passed command:

Re: Review Request 61849: Improved consistency of cout/cerr and glog usage in main functions.

2017-08-29 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61849/#review184072 --- Patch looks great! Reviews applied: [61849] Logs available

Re: Review Request 61849: Improved consistency of cout/cerr and glog usage in main functions.

2017-08-29 Thread Armand Grillet
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61849/ --- (Updated Aug. 29, 2017, 3:56 p.m.) Review request for mesos, Andrei Budnik and

Re: Review Request 61849: Improved consistency of cout/cerr and glog usage in main functions.

2017-08-29 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61849/#review184045 --- Fix it, then Ship it! Thanks for this cleanup. We should

Re: Review Request 61849: Improved consistency of cout/cerr and glog usage in main functions.

2017-08-28 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61849/#review183970 --- Patch looks great! Reviews applied: [61849] Logs available

Re: Review Request 61849: Improved consistency of cout/cerr and glog usage in main functions.

2017-08-28 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61849/#review183946 --- Patch looks great! Reviews applied: [61849] Passed command:

Re: Review Request 61849: Improved consistency of cout/cerr and glog usage in main functions.

2017-08-28 Thread Andrei Budnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61849/#review183930 --- Ship it! - Andrei Budnik On Aug. 25, 2017, 5:34 p.m., Armand

Re: Review Request 61849: Improved consistency of cout/cerr and glog usage in main functions.

2017-08-28 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61849/#review183925 --- Patch looks great! Reviews applied: [61849] Logs available

Re: Review Request 61849: Improved consistency of cout/cerr and glog usage in main functions.

2017-08-25 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61849/#review183886 --- Patch looks great! Reviews applied: [61849] Passed command:

Re: Review Request 61849: Improved consistency of cout/cerr and glog usage in main functions.

2017-08-25 Thread Armand Grillet
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61849/ --- (Updated Aug. 25, 2017, 5:34 p.m.) Review request for mesos, Andrei Budnik and

Re: Review Request 61849: Improved consistency of cout/cerr and glog usage in main functions.

2017-08-25 Thread Andrei Budnik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61849/#review183858 --- src/master/main.cpp Line 138 (original), 138 (patched)

Review Request 61849: Improved consistency of cout/cerr and glog usage in main functions.

2017-08-23 Thread Armand Grillet
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61849/ --- Review request for mesos, Andrei Budnik and Alexander Rukletsov. Bugs: