thrift-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Can Celasun (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (THRIFT-4652) Invalid Go source for Services
Date Wed, 17 Oct 2018 06:42:00 GMT

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

Can Celasun commented on THRIFT-4652:
-------------------------------------

> Do I have to build Apache Thrift from source to get this fix?

That commit is not a fix, you are just using incompatible versions of the compiler and library.
You are installing the compiler using brew, which correctly gives you the latest stable version
(0.11) but you got the _library_ from "go get" (or similar) which grabbed the latest master.
You have two options:
 * Build the compiler from master yourself (not recommended)
 * Use the 0.11 version of the library. If you are using {{dep}} for package management, the
command would be "dep ensure -add github.com/apache/thrift/lib/go/thrift@^0.11"

> Invalid Go source for Services
> ------------------------------
>
>                 Key: THRIFT-4652
>                 URL: https://issues.apache.org/jira/browse/THRIFT-4652
>             Project: Thrift
>          Issue Type: Bug
>          Components: Go - Compiler
>    Affects Versions: 0.11.0
>         Environment: macOS 10.14 
> Go 1.11.1 darwin/amd64
> Thrift 0.11.0
>  
>  
>            Reporter: Tristian Azura
>            Priority: Major
>
> The Go compiler generates invalid code for Thrift services, more specifically, calls
to the `thrift.TProtocol`'s `Flush()` method are missing the `context.Context` argument:
> {noformat}
> edam/NoteStore.go:18963:14: not enough arguments in call to oprot.Flush
> have ()
> want (context.Context)
> edam/NoteStore.go:18980:16: not enough arguments in call to oprot.Flush
> have ()
> want (context.Context)
> edam/NoteStore.go:18999:16: not enough arguments in call to oprot.Flush
> have ()
> want (context.Context)
> edam/NoteStore.go:19014:24: not enough arguments in call to oprot.Flush
> have ()
> want (context.Context)
> edam/NoteStore.go:19035:16: not enough arguments in call to oprot.Flush
> have ()
> want (context.Context)
> edam/NoteStore.go:19054:16: not enough arguments in call to oprot.Flush
> have ()
> want (context.Context)
> edam/NoteStore.go:19069:24: not enough arguments in call to oprot.Flush
> have ()
> want (context.Context)
> edam/NoteStore.go:19090:16: not enough arguments in call to oprot.Flush
> have ()
> want (context.Context)
> edam/NoteStore.go:19111:16: not enough arguments in call to oprot.Flush
> have ()
> want (context.Context)
> edam/NoteStore.go:19126:24: not enough arguments in call to oprot.Flush
> have ()
> want (context.Context){noformat}
>  
> I used the following models:
>  * [https://github.com/evernote/evernote-thrift]
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message