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

Chris Bannister edited comment on THRIFT-4285 at 8/18/17 10:05 AM:
-------------------------------------------------------------------

Fix typo, use tmp names for args/results incase the service defines the 
argument name as 'args' or 'result'.

This doesn't work with the read_write_private option as the type no longer 
implement TStruct, not sure what the use of that flag is given the rest of the 
read/write methods are public.


was (Author: zariel):
Fix typo, use tmp names for args/results incase the service defines the 
argument name as 'args' or 'result'.

> Pull generated send/recv into library to allow behaviour to be customised
> -------------------------------------------------------------------------
>
>                 Key: THRIFT-4285
>                 URL: https://issues.apache.org/jira/browse/THRIFT-4285
>             Project: Thrift
>          Issue Type: Improvement
>          Components: Go - Compiler, Go - Library
>            Reporter: Chris Bannister
>            Assignee: Chris Bannister
>         Attachments: 0001-go-pull-generated-send-recv-into-lib-2.patch, 
> 0001-go-pull-generated-send-recv-into-lib.patch, 
> 0001-go-pull-generated-send-recv-into-lib-v3.patch, 
> 0001-go-pull-generated-send-recv-into-lib-v4.patch
>
>
> Currently it is difficult to change how thrift writes messages onto the 
> transport because they are in the generated code. Instead the generated 
> send/recv methods should be in the library. This will greatly simplify the 
> client code and remove many duplicate methods whilst allowing users more 
> flexibility to implement connection pools and other features such as THeader.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to