----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32139/#review77771 -----------------------------------------------------------
include/mesos/mesos.proto <https://reviews.apache.org/r/32139/#comment125995> MPark, sorry if I leave the comment in a wrong place, I don't know where we discuss the design now. I'm sure you'll point me to the right doc : ). Q1: Do we allow frameworks to reserve dynamically for role, or just operators? Q2: If the answer to Q1 is "no", does it make sense to distinguish between static reservations and dynamic reservations since both of them are done by an operator? Q3: If the answer to Q2 is "yes", how are we going to distinguish between framework and operator dynamic *role* reservations if we introduce framework role reservations one day? include/mesos/mesos.proto <https://reviews.apache.org/r/32139/#comment125976> Let's insert a whitespace `"*" | R` if you are going to edit this. - Alexander Rukletsov On March 25, 2015, 6:47 p.m., Michael Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32139/ > ----------------------------------------------------------- > > (Updated March 25, 2015, 6:47 p.m.) > > > Review request for mesos, Alexander Rukletsov, Ben Mahler, and Jie Yu. > > > Bugs: MESOS-2475 > https://issues.apache.org/jira/browse/MESOS-2475 > > > Repository: mesos > > > Description > ------- > > The beginning of `Resource::ReservationInfo`. This patch only introduces the > `framework_id` field. We don't need the required `principal` field yet and > will be introduced together with the "unreserve" ACLs. An optional `id` may > be introduced in the future as well. > > > Diffs > ----- > > include/mesos/mesos.proto 3c592d5ab3092ecbeddfaff95e0c1addc3ac58f8 > > Diff: https://reviews.apache.org/r/32139/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Michael Park > >
