Re: Review Request 43237: Add benchmark for feching tasks.

2016-02-07 Thread Zameer Manji
> On Feb. 5, 2016, 11:05 a.m., Maxim Khutornenko wrote: > > src/jmh/java/org/apache/aurora/benchmark/TaskStoreBenchmarks.java, line 64 > > > > > > storeTasks or createTasks? Otherwise it's confusing to see loadTasks

Re: Review Request 43237: Add benchmark for feching tasks.

2016-02-07 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43237/ --- (Updated Feb. 7, 2016, 2:49 p.m.) Review request for Aurora, Maxim Khutornenko

Re: Review Request 43237: Add benchmark for feching tasks.

2016-02-07 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43237/#review118197 --- Ship it! Master (2d59b69) is green with this patch.

Re: Review Request 43237: Add benchmark for feching tasks.

2016-02-05 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43237/#review118058 --- Ship it!

Re: Review Request 43237: Add benchmark for feching tasks.

2016-02-05 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43237/ --- (Updated Feb. 5, 2016, 10:57 a.m.) Review request for Aurora, Maxim

Re: Review Request 43237: Add benchmark for feching tasks.

2016-02-05 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43237/#review118064 --- Ship it! Master (9b7c72e) is green with this patch.

Re: Review Request 43237: Add benchmark for feching tasks.

2016-02-04 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43237/ --- (Updated Feb. 4, 2016, 6:04 p.m.) Review request for Aurora, Maxim Khutornenko

Re: Review Request 43237: Add benchmark for feching tasks.

2016-02-04 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43237/#review117940 --- Master (505f310) is red with this patch.