mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Vinod Kone <vinodk...@gmail.com>
Subject Re: Review Request 53450: Added TTYInfo to ContainerInfo.
Date Fri, 04 Nov 2016 21:00:34 GMT


> On Nov. 4, 2016, 8:16 p.m., Kevin Klues wrote:
> > include/mesos/mesos.proto, line 2115
> > <https://reviews.apache.org/r/53450/diff/2/?file=1553918#file1553918line2115>
> >
> >     I see you changed s/TtyInfo/TTYInfo/
> >     
> >     Is this intentional? I know @bmahler has lamented in the past about decisions
like this. (e.g. `HTTPScheduler` he would prefer to be `HttpScheduler`).

I might be missing context but why do we prefer HttpScheduler to HTTPScheduler? HTTP is an
abbrevation, so all caps seems better to me. Note that we have TCPCheckInfo, HTTPCheckInfo,
URI, URL, etc protos already.


- Vinod


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53450/#review154980
-----------------------------------------------------------


On Nov. 4, 2016, 12:35 a.m., Vinod Kone wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/53450/
> -----------------------------------------------------------
> 
> (Updated Nov. 4, 2016, 12:35 a.m.)
> 
> 
> Review request for mesos, Anand Mazumdar, Benjamin Mahler, Jie Yu, and Kevin Klues.
> 
> 
> Bugs: MESOS-6525
>     https://issues.apache.org/jira/browse/MESOS-6525
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Added to ContainerInfo because we want to be able to attach to
> containers that are launched without a CommandInfo.
> 
> 
> Diffs
> -----
> 
>   include/mesos/mesos.proto 905a34bb13c2fcd1b22ef4e9605405ef84711c25 
>   include/mesos/v1/mesos.proto b72069a6caff0b163d9d9cae45c3caa0b25de38f 
> 
> Diff: https://reviews.apache.org/r/53450/diff/
> 
> 
> Testing
> -------
> 
> make
> 
> 
> Thanks,
> 
> Vinod Kone
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message