> On April 21, 2015, 12:51 a.m., Marco Massenzio wrote: > > src/master/http.cpp, line 300 > > <https://reviews.apache.org/r/30612/diff/12/?file=935462#file935462line300> > > > > can we have all routes patters as CONSTANTS defined in the class > > header? definitely easier to debug etc.
Marco, we tend not to use constants for non-POD types: https://issues.apache.org/jira/browse/MESOS-1023 An example of a prefered way is `MAX_REAP_INTERVAL()` from `reap.hpp`. In this particular case, why do you want to increase the visibility scope of this constant? - Alexander ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30612/#review80865 ----------------------------------------------------------- On April 20, 2015, 4:27 p.m., Alexander Rojas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30612/ > ----------------------------------------------------------- > > (Updated April 20, 2015, 4:27 p.m.) > > > Review request for mesos, Ben Mahler, Marco Massenzio, Niklas Nielsen, and > Till Toenshoff. > > > Bugs: MESOS-2157 > https://issues.apache.org/jira/browse/MESOS-2157 > > > Repository: mesos > > > Description > ------- > > Adds endpoints for paths /master/frameworks/{framework}/tasks/{task}. > > Adds tests for the endpoints. > > Works with [29883](https://reviews.apache.org/r/29883). > > Builds on the abandoned patch 14286. > > > Diffs > ----- > > src/master/http.cpp 00c22c43bd1f6cef7963b2ffa9c095c6cbd01cd3 > src/master/master.hpp c10e7c08c191acef9d31d98018a47a2a952a4dfc > src/master/master.cpp e30b951eda2b3b0d5b2a80716f0b32c6bbe041bc > src/tests/master_tests.cpp 32b1e9bb58d8046e5363fafe2ab8f662b6c9a666 > > Diff: https://reviews.apache.org/r/30612/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Alexander Rojas > >
