mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ben Mahler" <>
Subject Re: Review Request 36450: Introduced Address and URL protobufs.
Date Fri, 17 Jul 2015 01:36:50 GMT

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

(Updated July 17, 2015, 1:36 a.m.)

Review request for mesos, Benjamin Hindman, Jie Yu, and Vinod Kone.


Added a TODO to use process::URL equality.

Bugs: MESOS-3012

Repository: mesos


To make the API more consistent, we'd like to have a single way to express a network address.
Also would like a way to express an HTTP address (a URL), which may include a path prefix.

This also enables the message passing optimization in the scheduler driver when receiving
events, per MESOS-3012.

Diffs (updated)

  include/mesos/mesos.proto d2f482668e671b30f2586f6aae9c20132ab4d1e4 
  include/mesos/type_utils.hpp eb7fe2562cfcff52288d1c216425068d1ba551c0 
  src/common/type_utils.cpp 2ad5b4cbe324c83e81fd7df7430652f5c0a4e30f 
  src/master/master.cpp 082758ef54597ad32cf0d025c147f0f44dd11961 
  src/tests/master_tests.cpp 767c86cbde31eeb49d110d04bfb5a3eb5d469afc 



Modified the simplest test I could find for offers.


Ben Mahler

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