----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44004/#review120810 -----------------------------------------------------------
include/mesos/mesos.proto (line 1577) <https://reviews.apache.org/r/44004/#comment182312> In order to expose this field in state.json, I think you need to add the filed to the corresponding `model` function in src/common/http.cpp as well. We won't need this once we move to the new JSONIFY api, but I think we would need it in the immediate future. - Avinash sridharan On Feb. 25, 2016, 3:15 p.m., Qian Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44004/ > ----------------------------------------------------------- > > (Updated Feb. 25, 2016, 3:15 p.m.) > > > Review request for mesos, Avinash sridharan and Jie Yu. > > > Bugs: MESOS-4758 > https://issues.apache.org/jira/browse/MESOS-4758 > > > Repository: mesos > > > Description > ------- > > Add 'name' field into NetworkInfo. > > > Diffs > ----- > > include/mesos/mesos.proto 33f6b0838360b61db70a247e5d6dfb16af15aa06 > include/mesos/v1/mesos.proto 1b0e709e76f3f6b44ab0434c649c064e8866c8a1 > > Diff: https://reviews.apache.org/r/44004/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Qian Zhang > >
