Re: Review Request 33793: HTTP headers should be considered case-insensitive.

2015-05-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33793/ --- (Updated May 3, 2015, 10:41 a.m.) Review request for mesos and Ben Mahler.

Re: Review Request 30774: Fetcher Cache

2015-05-03 Thread Bernd Mathiske
On March 19, 2015, 9:40 a.m., Timothy Chen wrote: src/slave/containerizer/fetcher.cpp, line 379 https://reviews.apache.org/r/30774/diff/38/?file=899705#file899705line379 benh, what do you think of Bernd's contentionBarrier injection? commonly we always just mock the callback

Re: Review Request 30774: Fetcher Cache

2015-05-03 Thread Bernd Mathiske
On March 10, 2015, 8:35 a.m., Benjamin Hindman wrote: src/launcher/fetcher.cpp, lines 403-404 https://reviews.apache.org/r/30774/diff/33/?file=888350#file888350line403 As mentioned above, it would be great to really capture the relationship between the FetcherInfo and the

Review Request 33792: Add InsensitiveHashMap.

2015-05-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33792/ --- Review request for mesos and Ben Mahler. Bugs: MESOS-328

Review Request 33793: HTTP headers should be considered case-insensitive.

2015-05-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33793/ --- Review request for mesos and Ben Mahler. Bugs: MESOS-328

Review Request 33794: Update existing lambdas to meet style guide

2015-05-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33794/ --- Review request for mesos and Benjamin Hindman. Bugs: MESOS-2670

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

2015-05-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33794/ --- (Updated May 3, 2015, 10:31 a.m.) Review request for mesos and Benjamin

Re: Review Request 33792: Add InsensitiveHashMap.

2015-05-03 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33792/#review82338 --- Patch looks great! Reviews applied: [33792] All tests passed. -

Re: Review Request 33793: HTTP headers should be considered case-insensitive.

2015-05-03 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33793/#review82339 --- Bad patch! Reviews applied: [33793] Failed command: make -j3

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

2015-05-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33794/ --- (Updated May 3, 2015, 10:33 a.m.) Review request for mesos and Benjamin

Review Request 33795: Add a shutdownExecutor() method to the scheduler driver

2015-05-03 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33795/ --- Review request for mesos. Repository: mesos Description --- Add a