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

Hudson commented on THRIFT-3006:
--------------------------------

SUCCESS: Integrated in Thrift #1466 (See 
[https://builds.apache.org/job/Thrift/1466/])
THRIFT-3006 Attach 'omitempty' json tag for optional fields in Go (jensg: rev 
962e41078bb92656d4e28acfb717884e1ef57f5e)
* lib/go/test/GoTagTest.thrift
* lib/go/test/tests/gotag_test.go
* compiler/cpp/src/generate/t_go_generator.cc


> 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
>             Fix For: 0.9.3
>
>
> 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