----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38125/#review97749 -----------------------------------------------------------
src/master/master.cpp (line 3672) <https://reviews.apache.org/r/38125/#comment153766> Since you are also adding port to slaveID, so can you please also update the log message here by adding slaveInfo.port()? src/master/master.cpp (line 5846) <https://reviews.apache.org/r/38125/#comment153767> I think that we cannot make the assumpation here as the host FQDN name may include the underscore, so how to handle the csae of if slave name is FQDN name? What about using other charactors to connect? Such as # or others? http://stackoverflow.com/questions/10959757/the-use-of-the-underscore-in-host-names - Guangya Liu On 九月 4, 2015, 10:57 a.m., Bernd Mathiske wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38125/ > ----------------------------------------------------------- > > (Updated 九月 4, 2015, 10:57 a.m.) > > > Review request for mesos, Jan Schlicht and Till Toenshoff. > > > Bugs: MESOS-115 > https://issues.apache.org/jira/browse/MESOS-115 > > > Repository: mesos > > > Description > ------- > > Now including the hostname in the SlaveID, for human consumption. Also > testing this in a unit test. > > > Diffs > ----- > > src/master/master.hpp e1331851c19e3372a4a525dcfd7ba2a01c3e97a6 > src/master/master.cpp 5589eca4317b597de509f3387cfc349083b361ac > src/tests/master_tests.cpp 8a6b98b9f59ead20f537eb60b5084feed069a5b1 > > Diff: https://reviews.apache.org/r/38125/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Bernd Mathiske > >
