yuz10 commented on pull request #3386:
URL: https://github.com/apache/rocketmq/pull/3386#issuecomment-991408915


   > > The change is not compatible for go and cpp client, So the best way is 
add a new version of protocol.
   > 
   > The go and cpp client both use the standard json serializer and 
deserializer libraries. In fact, the go and cpp client have not implemented 
those complex invalid json-formatted requests and responses, so they have not 
faced this problem yet, but if they continue introducing new features, they 
will also be stuck in this problem at one day.
   
   I dont think so, The line 
https://github.com/apache/rocketmq-client-go/blob/master/internal/route.go#L533 
is not compatible with standard json.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to