----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45214/#review128751 -----------------------------------------------------------
include/mesos/mesos.proto (line 1625) <https://reviews.apache.org/r/45214/#comment192212> I definitely prefer this form, but there's also a Parameters message type. Which is more idiomatic in this context? - James DeFelice On April 13, 2016, 3:09 a.m., Guangya Liu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45214/ > ----------------------------------------------------------- > > (Updated April 13, 2016, 3:09 a.m.) > > > Review request for mesos, David vonThenen, Gilbert Song, haosdent huang, and > Jie Yu. > > > Bugs: MESOS-5012 > https://issues.apache.org/jira/browse/MESOS-5012 > > > Repository: mesos > > > Description > ------- > > Updated protobuf to support external storage. > > > Diffs > ----- > > include/mesos/mesos.proto 1232bcda053db3451d18413e81e54e51774f7f22 > include/mesos/v1/mesos.proto 1048fae62733926b5e863a0c1629bba69e35e43b > > Diff: https://reviews.apache.org/r/45214/diff/ > > > Testing > ------- > > make > make check > > > Thanks, > > Guangya Liu > >