[jira] [Resolved] (THRIFT-3150) Add an option to thrift go generator to make Read and Write methods private

["Jens Geyer (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/THRIFT-3150?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jens Geyer resolved THRIFT-3150.
--------------------------------
       Resolution: Fixed
    Fix Version/s: 0.9.3
         Assignee: Konstantin Shaposhnikov

Slightly modified & committed. 

I changed {{dont_export_rw}} into {{read_write_private}}, mostly because negative options {{dont_*}} are usually bad from an usability viewpoint.

> Add an option to thrift go generator to make Read and Write methods private
> ---------------------------------------------------------------------------
>
>                 Key: THRIFT-3150
>                 URL: https://issues.apache.org/jira/browse/THRIFT-3150
>             Project: Thrift
>          Issue Type: Improvement
>          Components: Go - Compiler
>    Affects Versions: 0.9.2
>            Reporter: Konstantin Shaposhnikov
>            Assignee: Konstantin Shaposhnikov
>             Fix For: 0.9.3
>
>
> I propose to add a new option *dont_export_rw* to go generator that if set will make it generate private read() and wirte() methods instead of public ones. E.g.:
> {code}
> func (p *Statistics) read(iprot thrift.TProtocol) error {
> ...
> }
> func (p *Statistics) write(oprot thrift.TProtocol) error {
> ....
> }
> {code}
> This will allow (along with THRIFT-3149) to generate a package that doesn't reference any symbols from Thrift Go library in public API.
> There are two main benefits:
> - keeping API of generated packages clean (only generated types are exported)
> - allowing to generate packages that can use vendored thrift library (e.g. placed inside internal/ directory).



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