Re: Review Request 32990: Improving JobUpdateDetails fetch performance.

2015-04-09 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32990/#review79562 --- Ship it! Ship It! - Bill Farner On April 9, 2015, 7:15 p.m.,

Re: Review Request 32990: Improving JobUpdateDetails fetch performance.

2015-04-09 Thread Maxim Khutornenko
On April 9, 2015, 2:29 a.m., Bill Farner wrote: Actually - can you add a benchmark to ThriftApiBenchmarks and include results in the description? Should be pretty easy to prove the improvement of this. Done. - Maxim --- This

Re: Review Request 32990: Improving JobUpdateDetails fetch performance.

2015-04-09 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32990/ --- (Updated April 9, 2015, 6:43 p.m.) Review request for Aurora and Bill Farner.

Re: Review Request 32990: Improving JobUpdateDetails fetch performance.

2015-04-09 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32990/#review79556 --- Master (7e3e7c9) is red with this patch.

Re: Review Request 32990: Improving JobUpdateDetails fetch performance.

2015-04-08 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32990/#review79441 --- Master (35f3e8d) is green with this patch.

Re: Review Request 32990: Improving JobUpdateDetails fetch performance.

2015-04-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32990/#review79466 --- Ship it! Great work! - Bill Farner On April 8, 2015, 9:45 p.m.,