Re: Review Request 61991: Added several logs to the C++ part of the v1-v0 adapter.

2017-09-01 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61991/ --- (Updated Sept. 1, 2017, 8:36 a.m.) Review request for mesos, Anand Mazumdar,

Re: Review Request 61991: Added several logs to the C++ part of the v1-v0 adapter.

2017-09-01 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61991/#review184338 --- Fix it, then Ship it! LGTM

Re: Review Request 61991: Added several logs to the C++ part of the v1-v0 adapter.

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

Re: Review Request 61991: Added several logs to the C++ part of the v1-v0 adapter.

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

Re: Review Request 61991: Added several logs to the C++ part of the v1-v0 adapter.

2017-08-31 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61991/ --- (Updated Aug. 31, 2017, 7:09 a.m.) Review request for mesos, Anand Mazumdar,

Re: Review Request 61991: Added several logs to the C++ part of the v1-v0 adapter.

2017-08-31 Thread Alexander Rukletsov
> On Aug. 30, 2017, 9:45 p.m., Anand Mazumdar wrote: > > src/java/jni/org_apache_mesos_v1_scheduler_V0Mesos.cpp > > Lines 360 (patched) > > > > > > hmm, I think we should only log things that are relevant to the >

Re: Review Request 61991: Added several logs to the C++ part of the v1-v0 adapter.

2017-08-30 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61991/#review184205 --- src/java/jni/org_apache_mesos_v1_scheduler_V0Mesos.cpp Lines 360

Re: Review Request 61991: Added several logs to the C++ part of the v1-v0 adapter.

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

Review Request 61991: Added several logs to the C++ part of the v1-v0 adapter.

2017-08-30 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61991/ --- Review request for mesos, Anand Mazumdar, Till Toenshoff, and Vinod Kone.