Re: Review Request 25133: Fixing race in task secondary index.

2014-08-28 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25133/#review51823 --- Ship it! Ship It! - Kevin Sweeney On Aug. 27, 2014, 6:19 p.m., M

Re: Review Request 25133: Fixing race in task secondary index.

2014-08-27 Thread Maxim Khutornenko
> On Aug. 28, 2014, 1:26 a.m., Kevin Sweeney wrote: > > src/test/java/org/apache/aurora/scheduler/storage/mem/MemTaskStoreTest.java, > > line 435 > > > > > > This looks like it adds a time dependency to our unit tests

Re: Review Request 25133: Fixing race in task secondary index.

2014-08-27 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25133/#review51736 --- src/test/java/org/apache/aurora/scheduler/storage/mem/MemTaskStoreT

Review Request 25133: Fixing race in task secondary index.

2014-08-27 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25133/ --- Review request for Aurora and Kevin Sweeney. Bugs: AURORA-667 https://issue