Re: Review Request 35096: Update mesos scheduler to use synchronized.

2015-06-13 Thread Benjamin Hindman

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35096/#review87822
---

Ship it!


Can also use synchronized with std::unique_lock here, I'll adjust and commit.

- Benjamin Hindman


On June 4, 2015, 10:54 p.m., Joris Van Remoortere wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/35096/
 ---
 
 (Updated June 4, 2015, 10:54 p.m.)
 
 
 Review request for mesos, Benjamin Hindman and Bernd Mathiske.
 
 
 Bugs: MESOS-2805
 https://issues.apache.org/jira/browse/MESOS-2805
 
 
 Repository: mesos
 
 
 Description
 ---
 
 See summary.
 
 
 Diffs
 -
 
   include/mesos/scheduler.hpp 2ee6b5cc0745b3ac4637bb97843a00024fb94570 
   src/sched/sched.cpp 8c366ec3e3cf55dacf49483e1ceeef61ab0187b3 
 
 Diff: https://reviews.apache.org/r/35096/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Joris Van Remoortere
 




Review Request 35096: Update mesos scheduler to use synchronized.

2015-06-04 Thread Joris Van Remoortere

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35096/
---

Review request for mesos, Benjamin Hindman and Bernd Mathiske.


Bugs: MESOS-2805
https://issues.apache.org/jira/browse/MESOS-2805


Repository: mesos


Description
---

See summary.


Diffs
-

  include/mesos/scheduler.hpp 2ee6b5cc0745b3ac4637bb97843a00024fb94570 
  src/sched/sched.cpp 8c366ec3e3cf55dacf49483e1ceeef61ab0187b3 

Diff: https://reviews.apache.org/r/35096/diff/


Testing
---


Thanks,

Joris Van Remoortere