Re: Review Request 32013: Cache TaskDetectors for reuse.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32013/#review76822 --- Ship it! Ship It! - Zameer Manji On March 17, 2015, 1:24 p.m., B

Re: Review Request 32013: Cache TaskDetectors for reuse.

> On March 17, 2015, 7:49 p.m., Zameer Manji wrote: > > src/main/python/apache/aurora/executor/common/path_detector.py, line 34 > > > > > > What's the rationale behind this change? I think the callers should be > > doi

Re: Review Request 32013: Cache TaskDetectors for reuse.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32013/#review76800 --- Ship it! Master (4b39d26) is green with this patch. ./build-suppo

Re: Review Request 32013: Cache TaskDetectors for reuse.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32013/#review76794 --- Ship it! Ship It! - Joe Smith On March 17, 2015, 1:24 p.m., Bria

Re: Review Request 32013: Cache TaskDetectors for reuse.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32013/ --- (Updated March 17, 2015, 8:24 p.m.) Review request for Aurora, Joe Smith and Za

Re: Review Request 32013: Cache TaskDetectors for reuse.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32013/#review76784 --- src/main/python/apache/aurora/executor/common/path_detector.py

Re: Review Request 32013: Cache TaskDetectors for reuse.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32013/#review76667 --- This patch does not apply cleanly on master (1b2a343), do you need t

Re: Review Request 32013: Cache TaskDetectors for reuse.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32013/ --- (Updated March 16, 2015, 10:37 p.m.) Review request for Aurora, Joe Smith and Z

Re: Review Request 32013: Cache TaskDetectors for reuse.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32013/#review76658 --- Master (1b2a343) is red with this patch. ./build-support/jenkins/b

Re: Review Request 32013: Cache TaskDetectors for reuse.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32013/ --- (Updated March 16, 2015, 9:47 p.m.) Review request for Aurora, Joe Smith and Za

Re: Review Request 32013: Cache TaskDetectors for reuse.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32013/#review76319 --- whoops, forgot that I added os.path.exists filtering -- will update

Re: Review Request 32013: Cache TaskDetectors for reuse.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32013/#review76318 --- Master (ee1a13a) is red with this patch. ./build-support/jenkins/b

Review Request 32013: Cache TaskDetectors for reuse.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32013/ --- Review request for Aurora, Joe Smith and Zameer Manji. Repository: aurora Des