Re: Review Request 51993: Added the 'reason' to the /pendingTasks endpoint

2016-11-28 Thread Pradyumna Kaushik
> On Nov. 28, 2016, 4:03 p.m., Joshua Cohen wrote: > > Sorry for the delay on this, everything looks good to me, just a few > > style/maintainability nits, then it's good to go! @jcohen. I have made the necessary changes and have posted the patch for review. Thank you. - Pradyumna

Re: Review Request 51993: Added the 'reason' to the /pendingTasks endpoint

2016-11-28 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51993/#review157132 --- Ship it! Master (d56f8c6) is green with this patch.

Re: Review Request 51993: Added the 'reason' to the /pendingTasks endpoint

2016-11-28 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51993/#review157136 --- Ship it! Ship It! - Joshua Cohen On Nov. 28, 2016, 8:34

Re: Review Request 54106: SlaUtil::percentile percentiles interpolation

2016-11-28 Thread Reza Motamedi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54106/ --- (Updated Nov. 28, 2016, 7:22 p.m.) Review request for Aurora, David

Re: Review Request 54106: SlaUtil::percentile percentiles interpolation

2016-11-28 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54106/#review157123 --- Master (d56f8c6) is red with this patch.

Re: Review Request 54107: changes to intercept and time mybatis invocations

2016-11-28 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54107/#review157154 --- Ship it! Master (8e07b04) is green with this patch.

Re: Review Request 54107: changes to intercept and time mybatis invocations

2016-11-28 Thread Reza Motamedi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54107/ --- (Updated Nov. 28, 2016, 10:10 p.m.) Review request for Aurora, David

Re: Review Request 54107: changes to intercept and time mybatis invocations

2016-11-28 Thread Reza Motamedi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54107/ --- (Updated Nov. 29, 2016, 2:36 a.m.) Review request for Aurora, David

Re: Review Request 54107: changes to intercept and time mybatis invocations

2016-11-28 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54107/#review157195 --- Ship it! Master (8e07b04) is green with this patch.