This is an automatically generated e-mail. To reply, visit:

(Updated April 14, 2016, 3:34 p.m.)

Review request for mesos and Vinod Kone.


Review comments from Vinod

Bugs: MESOS-3923

Repository: mesos

Description (updated)

Use the `principal` set in `FrameworkInfo.principal` to populate
framework metrics instead of looking them up in `authenticated`
map. It also ensures that HTTP->PID framework downgrades preserve 
the metrics and no additional logic is needed to deal with this. 
(Previously HTTP frameworks never had AuthN and no principal was 
passed in `FrameworkInfo`, so this was never a concern).

Diffs (updated)

  src/master/master.cpp 781402c04fded159183e1ca28894e48355200f0c 

Diff: https://reviews.apache.org/r/46114/diff/


make check


Anand Mazumdar

Reply via email to