[ 
https://issues.apache.org/jira/browse/MESOS-9901?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16891171#comment-16891171
 ] 

Benjamin Mahler commented on MESOS-9901:
----------------------------------------

[~bbannier] hm.. not sure how the existing format was produced but it doesn't 
comply with the standard mapping?

https://developers.google.com/protocol-buffers/docs/proto3#json

I think we should just bite the bullet and send out an email to make the 
breaking change to get towards the proto3 standard json mapping.

> Specialize jsonify for protobuf Maps.
> -------------------------------------
>
>                 Key: MESOS-9901
>                 URL: https://issues.apache.org/jira/browse/MESOS-9901
>             Project: Mesos
>          Issue Type: Improvement
>          Components: json api
>            Reporter: Meng Zhu
>            Priority: Major
>
> Jsonify current treats protobuf as a regular repeated field. For example, for 
> the schema 
> {noformat}
> message QuotaConfig {
>   required string role = 1;
>   map<string, Value.Scalar> guarantees = 2;
>   map<string, Value.Scalar> limits = 3;
> }
> {noformat}
> it will produce:
> {noformat}
>       "configs": [
>         {
>           "role": "role1",
>           "guarantees": [
>             {
>               "key": "cpus",
>               "value": {
>                 "value": 1
>               }
>             },
>             {
>               "key": "mem",
>               "value": {
>                 "value": 512
>               }
>             }
>           ]
> {noformat}
> This output cannot be parsed back to proto messages. We need to specialize 
> jsonify for Maps type. 



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

Reply via email to