Re: Review Request 34645: Update existing lambdas to meet style guide

2015-06-09 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34645/#review87149 --- Ship it! Ship It! - Benjamin Hindman On June 1, 2015, 5:26

Re: Review Request 34645: Update existing lambdas to meet style guide

2015-06-01 Thread haosdent huang
On May 25, 2015, 11:05 p.m., Joris Van Remoortere wrote: This is an interesting case. We have a proxy to another function, rather than the implementation of that function as a lambda. I'm curious what the community's view is on using the proxy lambda approach as per your patch, versus

Re: Review Request 34645: Update existing lambdas to meet style guide

2015-06-01 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34645/ --- (Updated June 1, 2015, 5:24 p.m.) Review request for mesos, Benjamin Hindman

Re: Review Request 34645: Update existing lambdas to meet style guide

2015-06-01 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34645/ --- (Updated June 1, 2015, 5:26 p.m.) Review request for mesos, Benjamin Hindman

Re: Review Request 34645: Update existing lambdas to meet style guide

2015-06-01 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34645/#review86056 --- Patch looks great! Reviews applied: [34644, 34645] All tests

Re: Review Request 34645: Update existing lambdas to meet style guide

2015-05-29 Thread haosdent huang
On May 25, 2015, 11:05 p.m., Joris Van Remoortere wrote: This is an interesting case. We have a proxy to another function, rather than the implementation of that function as a lambda. I'm curious what the community's view is on using the proxy lambda approach as per your patch, versus

Re: Review Request 34645: Update existing lambdas to meet style guide

2015-05-25 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34645/#review85137 --- This is an interesting case. We have a proxy to another function,

Review Request 34645: Update existing lambdas to meet style guide

2015-05-24 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34645/ --- Review request for mesos, Benjamin Hindman and Joris Van Remoortere. Bugs: