Re: Review Request 35498: Compute SLA stats for non-prod jobs

2015-06-17 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35498/#review88261 --- src/main/java/org/apache/aurora/scheduler/sla/SlaAlgorithm.java (li

Review Request 35580: Fix symlink duplicates from MesosPathDetector.

2015-06-17 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35580/ --- Review request for Aurora and Maxim Khutornenko. Bugs: AURORA-1353 https://

Re: Review Request 35580: Fix symlink duplicates from MesosPathDetector.

2015-06-17 Thread Joe Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35580/#review88278 --- Ship it! +1 on test coverage too - Joe Smith On June 17, 2015, 2

Re: Review Request 35580: Fix symlink duplicates from MesosPathDetector.

2015-06-17 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35580/#review88281 --- Ship it! Ship It! - Maxim Khutornenko On June 17, 2015, 9:17 p.m

Review Request 35587: Suppress task reconciliation status update logging.

2015-06-17 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35587/ --- Review request for Aurora and Bill Farner. Repository: aurora Description ---

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. ./build-suppo