Re: Review Request 35587: Suppress task reconciliation status update logging.

2015-06-22 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35587/#review88806 --- Ship it! Ship It! - Bill Farner On June 19, 2015, 10:07 p.m.,

Re: Review Request 35587: Suppress task reconciliation status update logging.

2015-06-19 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35587/ --- (Updated June 19, 2015, 10:07 p.m.) Review request for Aurora and Bill Farner.

Re: Review Request 35587: Suppress task reconciliation status update logging.

2015-06-19 Thread Maxim Khutornenko
On June 18, 2015, 4:42 p.m., Bill Farner wrote: src/test/java/org/apache/aurora/scheduler/mesos/MesosSchedulerImplTest.java, line 314 https://reviews.apache.org/r/35587/diff/1/?file=986660#file986660line314 What's being tested here? If i'm reading correctly, this test case

Re: Review Request 35587: Suppress task reconciliation status update logging.

2015-06-18 Thread Bill Farner
On June 18, 2015, 4:42 p.m., Bill Farner wrote: src/test/java/org/apache/aurora/scheduler/mesos/MesosSchedulerImplTest.java, line 314 https://reviews.apache.org/r/35587/diff/1/?file=986660#file986660line314 What's being tested here? If i'm reading correctly, this test case

Re: Review Request 35587: Suppress task reconciliation status update logging.

2015-06-18 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35587/ --- (Updated June 18, 2015, 5:04 p.m.) Review request for Aurora and Bill Farner.

Re: Review Request 35587: Suppress task reconciliation status update logging.

2015-06-18 Thread Maxim Khutornenko
On June 18, 2015, 4:42 p.m., Bill Farner wrote: src/test/java/org/apache/aurora/scheduler/mesos/MesosSchedulerImplTest.java, line 314 https://reviews.apache.org/r/35587/diff/1/?file=986660#file986660line314 What's being tested here? If i'm reading correctly, this test case

Re: Review Request 35587: Suppress task reconciliation status update logging.

2015-06-18 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35587/#review88388 ---

Re: Review Request 35587: Suppress task reconciliation status update logging.

2015-06-18 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35587/#review88415 --- Ship it! Master (af7e1a7) is green with this patch.

Re: Review Request 35587: Suppress task reconciliation status update logging.

2015-06-17 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35587/#review88325 --- Ship it! Master (af7e1a7) is green with this patch.