go-thrift icon indicating copy to clipboard operation
go-thrift copied to clipboard

A native Thrift package for Go

Results 39 go-thrift issues
Sort by recently updated
recently updated
newest added

cpp_include now causes syntax error

Hi there, I made [a PR](https://github.com/samuel/go-thrift/pull/103) recently that updated some function comments. This one will make [CodeLingo](https://www.codelingo.io/) review your PRs and find similar issues automatically. It will find issues from...

Is this project abandoned? There seem to be an awful lot of issues with no responses and PR's that have been open for many months now..

Example: ``` struct Foo { 1: optional list Bar = [ "hello", "world" ] } ``` Returns: ``` $ generator example.thrift $GOPATH/src example.thrift:1:1 (0): rule SyntaxError: parser: syntax error ```...

Example: ``` const map Foo = { "hello": "it's me" "hi": "how are you", } ``` Returns: ``` $ generator example.thrift $GOPATH/src example.thrift:1:1 (0): rule SyntaxError: parser: syntax error ```...

Go packages can't be nested but you can nest namespaces in Thrift. If you have two Thrift files with the same "leaf" namespace name then when both namespaces are used...

* Normalizes as Required for field name vs Optional * Adds tag 'implicit' when not declared. * Don't error when decoding if a required field is implicit. Reference: https://thrift.apache.org/docs/idl#field-requiredness >...

IDL: service Calculator extends shared.SharedService is generate GO: type Calculator interface { Shared.SharedService Add(num1 int32, num2 int32) (int32, error) Calculate(logid int32, w *Work) (int32, error) Ping() error } this Shared.SharedService...

Currently a uint64 is used to track required fields. This limits a struct to fields with ID < 64. Ideally this limit should be removed.

This is broken out into logical commits, tests pass after each commit (so you could choose to accept some subset). Commit messages should be explanatory. I'm happy to help split...