Re: Review Request 35672: DbTaskStore perf: optimize queries scoped to a task ID.

2015-06-22 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35672/#review88804 --- Ship it! Ship It! - Maxim Khutornenko On June 19, 2015, 10:43

Re: Review Request 35672: DbTaskStore perf: optimize queries scoped to a task ID.

2015-06-22 Thread Maxim Khutornenko
On June 19, 2015, 11:50 p.m., Maxim Khutornenko wrote: src/main/java/org/apache/aurora/scheduler/storage/TaskStore.java, line 127 https://reviews.apache.org/r/35672/diff/1/?file=988557#file988557line127 Curious, why special treating whitespacing here and not in other places? Bill

Re: Review Request 35672: DbTaskStore perf: optimize queries scoped to a task ID.

2015-06-22 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35672/ --- (Updated June 22, 2015, 4:51 p.m.) Review request for Aurora and Maxim

Re: Review Request 35672: DbTaskStore perf: optimize queries scoped to a task ID.

2015-06-19 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35672/#review88600 --- Master (d240926) is green with this patch.

Re: Review Request 35672: DbTaskStore perf: optimize queries scoped to a task ID.

2015-06-19 Thread Bill Farner
On June 19, 2015, 11:50 p.m., Maxim Khutornenko wrote: src/main/java/org/apache/aurora/scheduler/storage/TaskStore.java, line 127 https://reviews.apache.org/r/35672/diff/1/?file=988557#file988557line127 Curious, why special treating whitespacing here and not in other places? This