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

ASF GitHub Bot commented on THRIFT-3006:
----------------------------------------

Github user pjjw commented on the pull request:

    https://github.com/apache/thrift/pull/380#issuecomment-75808763
  
    @Magrath branch name needs to, which it does. see 
https://issues.apache.org/jira/browse/THRIFT-3006 for upstream.


> Attach 'omitempty' json tag for optional fields in Go
> -----------------------------------------------------
>
>                 Key: THRIFT-3006
>                 URL: https://issues.apache.org/jira/browse/THRIFT-3006
>             Project: Thrift
>          Issue Type: Improvement
>          Components: Go - Compiler
>            Reporter: peter woodman
>            Assignee: Jens Geyer
>
> This pull request improves on THRIFT-2549 in that it additionally tags 
> optional fields with the omitempty tag so that dumping a struct to json 
> doesn't result in a bunch of null fields. I think this is the right thing to 
> do with optional fields, and additionally it matches the protoc behavior in 
> go.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to