thrift-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (THRIFT-4285) Pull generated send/recv into library to allow behaviour to be customised
Date Tue, 10 Oct 2017 12:57:00 GMT

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

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

Github user dcelasun commented on the issue:

    https://github.com/apache/thrift/pull/1382
  
    > Is there a way you can provide a generated NewFooClientFactory as an adapter to run
the new code?
    
    It would require putting back some of the generated code, but I'll try to find a way.
I agree avoiding the BC break is worth it if possible.
    
    > I was away for a couple days on college tours with my son, sorry for the delay.
    
    No worries, I just wanted to ping you in case this slipped through the cracks.


> 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-v6.patch, 0001-go-pull-generated-send-recv-into-lib-v7.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)

Mime
View raw message