----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56901/ -----------------------------------------------------------
(Updated March 3, 2017, 6:41 p.m.) Review request for mesos, Adam B, Alexander Rojas, Jan Schlicht, Till Toenshoff, and Vinod Kone. Changes ------- Adjusted code for the `principal->value.isSome()` invariant. Bugs: MESOS-7003 https://issues.apache.org/jira/browse/MESOS-7003 Repository: mesos Description ------- This patch updates master validation code to make use of the `Principal` type instead of an `Option<string> principal`. Diffs (updated) ----- src/master/validation.hpp 50a3e6ec6a6d44c9ad449edfbd5d2cf8540b3a73 src/master/validation.cpp 41ef0d072ca53d9695c16b8dd98c319186f11f75 Diff: https://reviews.apache.org/r/56901/diff/4/ Changes: https://reviews.apache.org/r/56901/diff/3-4/ Testing ------- Testing details can be found at the end of this review chain. Thanks, Greg Mann