----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54200/#review158330 -----------------------------------------------------------
Patch looks great! Reviews applied: [54359, 54360, 54361, 54365, 54200] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker_build.sh - Mesos ReviewBot On Dec. 7, 2016, 8:59 a.m., Jay Guo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54200/ > ----------------------------------------------------------- > > (Updated Dec. 7, 2016, 8:59 a.m.) > > > Review request for mesos, Benjamin Mahler, Guangya Liu, and Qian Zhang. > > > Bugs: MESOS-6635 > https://issues.apache.org/jira/browse/MESOS-6635 > > > Repository: mesos > > > Description > ------- > > The function returns a set<string> containing all roles in a > multi-role FrameworkInfo. For single-role FrameworkInfo, there > is only one element in the set. > > > Diffs > ----- > > src/common/protobuf_utils.hpp 80c66c4f7d52fc184e7553d173195ba7714971d4 > src/common/protobuf_utils.cpp 56e16648ae138ed6da40811144b721cd82b5f8a0 > > Diff: https://reviews.apache.org/r/54200/diff/ > > > Testing > ------- > > > Thanks, > > Jay Guo > >
