Re: Review Request 35752: Added stub Event protobuf handler to scheduler driver.

2015-06-23 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35752/#review89099 --- src/sched/sched.cpp (lines 462 - 464)

Re: Review Request 35752: Added stub Event protobuf handler to scheduler driver.

2015-06-23 Thread Till Toenshoff
On June 23, 2015, 11:31 p.m., Isabel Jimenez wrote: src/sched/sched.cpp, line 438 https://reviews.apache.org/r/35752/diff/1/?file=990748#file990748line438 Aren't we using braces on switch cases syntax? like: ``` switch (type) { case ENUM: { break

Re: Review Request 35752: Added stub Event protobuf handler to scheduler driver.

2015-06-23 Thread Isabel Jimenez
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35752/#review89085 --- src/sched/sched.cpp (line 438)