Re: Review Request 63383: Suppress multiline logging from mesos callbacks

2017-10-28 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63383/#review189508 --- This has been submitted already

Re: Review Request 63383: Suppress multiline logging from mesos callbacks

2017-10-27 Thread Joshua Cohen
> On Oct. 28, 2017, 12:08 a.m., Jordan Ly wrote: > > src/main/java/org/apache/aurora/scheduler/mesos/MesosCallbackHandler.java > > Lines 303-305 (patched) > > > > > > Does Mesos send any other important information

Re: Review Request 63383: Suppress multiline logging from mesos callbacks

2017-10-27 Thread Bill Farner
> On Oct. 27, 2017, 5:08 p.m., Jordan Ly wrote: > > src/main/java/org/apache/aurora/scheduler/mesos/MesosCallbackHandler.java > > Lines 303-305 (patched) > > > > > > Does Mesos send any other important information

Re: Review Request 63383: Suppress multiline logging from mesos callbacks

2017-10-27 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63383/#review189498 --- Ship it! Master (f6c40a2) is green with this patch.

Re: Review Request 63383: Suppress multiline logging from mesos callbacks

2017-10-27 Thread Jordan Ly
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63383/#review189497 --- Fix it, then Ship it! One small question, but overall LGTM.