Github user qianzhangxa commented on the pull request:

    
https://github.com/apache/mesos/commit/5b2261409e2b391d9182c5579e1df481c7f0779f#commitcomment-28615233
  
    @shuizz `protobuf::internal::parse(google::protobuf::Message* message, 
const JSON::Object& object)` is used to convert a JSON object to a protobuf 
message, can you please show me your JSON which caused the error: `Not 
expecting a JSON string for field 'value' `.
    
    Actually, we already have a [unit 
test](https://github.com/apache/mesos/blob/master/3rdparty/stout/tests/protobuf_tests.cpp#L803:L806)
 to verify the case that the value of map is a protobuf message, and I see it 
works well now.


---

Reply via email to