> On Nov. 14, 2017, 10:12 p.m., Gaston Kleiman wrote: > > src/scheduler/scheduler.cpp > > Lines 123-124 (original), 123 (patched) > > <https://reviews.apache.org/r/63768/diff/3/?file=1892143#file1892143line123> > > > > For consistency I think that it'd also be nice to remove the `using > > process::http::Request` statement. > > > > Also I noticed that we use `::Request` in this file... so removing the > > `using` statement and replacing `::Request` with the more explicit > > `process::http::Request` feels to me like an improvement.
Good call; I added another patch to do this: https://reviews.apache.org/r/63850/ - Greg ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63768/#review191007 ----------------------------------------------------------- On Nov. 14, 2017, 9:44 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63768/ > ----------------------------------------------------------- > > (Updated Nov. 14, 2017, 9:44 p.m.) > > > Review request for mesos, Benjamin Bannier, Gaston Kleiman, Jie Yu, and Jan > Schlicht. > > > Bugs: MESOS-8198 > https://issues.apache.org/jira/browse/MESOS-8198 > > > Repository: mesos > > > Description > ------- > > This patch adds a first-class `Response` message to the scheduler API > along with its first member, the `ReconcileOfferOperations` response > message. > > > Diffs > ----- > > include/mesos/scheduler/scheduler.proto > f5dda10a076ecae53a56875d0ed4d201a1c92f12 > include/mesos/v1/scheduler/scheduler.proto > 4fa6606b0db701aebe208b962b7768da68b180ab > src/scheduler/scheduler.cpp 286a7603352a1e639a0502e496d4409be11bf9d7 > > > Diff: https://reviews.apache.org/r/63768/diff/3/ > > > Testing > ------- > > make check > > > Thanks, > > Greg Mann > >
