> On Sept. 12, 2016, 10:57 a.m., Jiang Yan Xu wrote: > > include/mesos/mesos.proto, line 372 > > <https://reviews.apache.org/r/51803/diff/3/?file=1496755#file1496755line372> > > > > In genenal I think we should state "Feature X will be deprecated in > > version Y in favor of feature Z" to help folks understand the reasoning. In > > this case there happens to be difference of opinions on whether this > > feature/field should be deprecated in 2.0 (which we can discuss outside > > this review) but could you at least in this review add why this is > > deprecated and what do you recommend people to do when it is deprecated? > > haosdent huang wrote: > @xujyan, that requires we define the message to support HTTP health check > with statues. How about let me rephrase the comment here to describe it is > not supported and may be deprecated in Mesos 2.0?
I think the following statement is correct for 1.x, right? If so can we use it? ``` Expected response statuses. NOTE: It is up to the custom executor to interpret and act on this field. The default executors don't use this field and setting this field has no effect on them (they interpret 2xx - 3xx as healthy). ``` I understand that we are considering changing this but: - If it's unclear what we are going to do about it, is it worth forewarning people about it going away? i.e., are you sure in the new design it's going away? - Do we know if this improvement is scheduled for after 2.0? i.e., not in 1.x? - We already have the TODO above, wouldn't it serve as a reminder that we'll address this instead of a warning (which may unnecessarily freak people out about an improvement that *may* not result in backwards-incompatible API change? - Jiang Yan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51803/#review148536 ----------------------------------------------------------- On Sept. 12, 2016, 11:30 a.m., haosdent huang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51803/ > ----------------------------------------------------------- > > (Updated Sept. 12, 2016, 11:30 a.m.) > > > Review request for mesos, Alexander Rukletsov, Joseph Wu, Silas Snider, and > Jiang Yan Xu. > > > Bugs: MESOS-6110 > https://issues.apache.org/jira/browse/MESOS-6110 > > > Repository: mesos > > > Description > ------- > > Ensured `HealthCheck::HTTPCheckInfo` compatible with the old one. > > > Diffs > ----- > > include/mesos/mesos.proto eb61b6243202464da2307d06d80700f19f9c25d4 > include/mesos/v1/mesos.proto 62231522f4bfddfc6c440a299dd01080cbe25f6a > > Diff: https://reviews.apache.org/r/51803/diff/ > > > Testing > ------- > > > Thanks, > > haosdent huang > >