mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Mahler <>
Subject Re: Review Request 71748: Support jsonifying v0 protobuf to v1 protobuf.
Date Fri, 22 Nov 2019 18:16:23 GMT

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

(Updated Nov. 22, 2019, 6:16 p.m.)

Review request for mesos, Andrei Sekretenko, Greg Mann, and Meng Zhu.


Split out the lower and upper cases.

Bugs: MESOS-10026

Repository: mesos


This allows us to jsonify a v0 protobuf directly to a v1 protobuf
efficiently, with no need to `evolve()` the message (which is rather

The way this works is by converting all "slave" and "SLAVE" strings
in fields and enum values, respectively, to "agent" and "AGENT".

Our current v0 to v1 conversion for the v1 operator API simply
serializes the v0 message and de-serializes into a v1 message, which
means all field tags and message structures are the same, except
for field names. The only difference with field names is the use
of "agent" in place of "slave".

Diffs (updated)

  src/common/http.hpp 062586c0a5a339c7e63c89a3a893ae015d3fd26e 
  src/common/http.cpp b79074f823d3bce2a15736c0ef4739ad13db8d9c 




Added a test in the subsequent patch.


Benjamin Mahler

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