[MESOS-7656] Update the JSON <=> protobuf message conversion for map support

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.6.0
• Target Version/s: 1.6.0
• Sprint: Mesosphere Sprint 76
• Story Points: 3
• Assignee: Qian Zhang
• Reporter: Qian Zhang
• Shepherd: Benjamin Mahler
• Created: 13/Jun/17 07:49
• Updated: 21/Mar/18 02:30
• Resolved: 14/Mar/18 03:36

Description

Map is a feature starting from proto2 syntax, but it can only be compiled with proto3 compiler, see the following discussion for details:
https://groups.google.com/forum/#&#33;topic/protobuf/p4WxcplrlA4
We have already upgraded the protobuf compiler from 2.6.1 to 3.3.0 in MESOS-7228, however, to use protobuf map in Mesos code, we also need to add the protobuf map support to the code in Mesos for converting protobuf message to JSON object and parsing JSON object as protobuf message, that is what we plan to handle in this ticket.

With map support added, the following field in the protobuf message can be replaced by the native protobuf map field, and we do not have to manually parse it anymore.
https://github.com/apache/mesos/blob/1.3.0/include/mesos/docker/v1.proto#L68

Issue Links

• is blocked by: MESOS-7228 Upgrade Mesos to build with proto3.
• is depended upon by: MESOS-8702 Replace the manual parsing in Mesos code with the native protobuf map support