----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28655/#review65388 -----------------------------------------------------------
Ship it! This looks great Kapil! src/Makefile.am <https://reviews.apache.org/r/28655/#comment108490> I would try to adhere more towards the backslash spacing as used by e.g. "logging/logging.hpp" and fix the others in this section (at least). AFAIK for Makefile.am we use 8 char hard-tabs. src/hook/manager.cpp <https://reviews.apache.org/r/28655/#comment108493> Is this going to be a growing list of checks for specific hook modules and add them to the respective hook interfaces? src/master/flags.hpp <https://reviews.apache.org/r/28655/#comment108489> Isnt lowercase "comma" just fine? - Till Toenshoff On Dec. 11, 2014, 11:23 p.m., Kapil Arya wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28655/ > ----------------------------------------------------------- > > (Updated Dec. 11, 2014, 11:23 p.m.) > > > Review request for mesos and Niklas Nielsen. > > > Repository: mesos-git > > > Description > ------- > > This hook allows hook modules to add additional labels to the incoming > TaskInfo object. The labels are then read on the slave/executor side which > may then act upon them. > > TODOs before we commit: > 0. Split this RR to move Master specific code to a separate RR. > 1. Add a test hook module. > > > Diffs > ----- > > src/Makefile.am 86161fe7a8bdd86958d24adb74d434cd92d7dfb8 > src/hook/hook.hpp PRE-CREATION > src/hook/manager.hpp PRE-CREATION > src/hook/manager.cpp PRE-CREATION > src/master/flags.hpp 1cea50c02f3ad7de1e1ae91d65d1accdb9af7b03 > src/master/main.cpp 193d53f13d8b14638b311cc290b5a5802ea56299 > src/master/master.cpp 1cf2074b78e260bcccf96f4383bc4747b1e75063 > src/module/hook.hpp PRE-CREATION > src/module/manager.cpp b15b0fc3f056fe29bd4d1acca508d75805ef2e0b > > Diff: https://reviews.apache.org/r/28655/diff/ > > > Testing > ------- > > > Thanks, > > Kapil Arya > >
