Re: Review Request 34739: Adding ChainedPathDetector into the GC executor.

2015-05-28 Thread Maxim Khutornenko
On May 28, 2015, 5 p.m., Brian Wickman wrote: src/test/python/apache/aurora/executor/bin/test_gc_executor_entry_point.py, line 42 https://reviews.apache.org/r/34739/diff/1/?file=973208#file973208line42 we tend to do assert mock.mock_calls == [mock.call(...)] to guard against

Re: Review Request 34739: Adding ChainedPathDetector into the GC executor.

2015-05-28 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34739/#review85568 --- Ship it! feel free to ship as-is. provided an alternate

Re: Review Request 34739: Adding ChainedPathDetector into the GC executor.

2015-05-27 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34739/#review85479 --- Ship it! Master (5724099) is green with this patch.