mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Greg Mann <>
Subject Re: Review Request 42733: Added checks for presence of `ReservationInfo.principal`.
Date Fri, 26 Feb 2016 22:21:29 GMT

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

(Updated Feb. 26, 2016, 10:21 p.m.)

Review request for mesos, Michael Park, Neil Conway, and Vinod Kone.


Addressed comment.

Bugs: MESOS-3940

Repository: mesos


Added checks for presence of `ReservationInfo.principal`.

The `ReservationInfo.principal` field was recently made `optional`. There remain a couple
places in the code that assume this field is always set. This patch uses `has_principal()`
before this field is accessed to ensure safety.

Diffs (updated)

  src/common/resources.cpp 5d731870542166cec11f9956ccdf16207b2d22cc 
  src/master/master.cpp 8d6d3c6468c6b85fe09c33cf9747cc3d1f515ab9 
  src/v1/resources.cpp 207eb61d6a6d03d314539d42751cac65fcffa9af 



`make check`


Greg Mann

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