[GitHub] thrift pull request: Thrift 3251 Add http transport for server to ...

2016-01-10 Thread Jens-G
Github user Jens-G commented on the pull request: https://github.com/apache/thrift/pull/557#issuecomment-170335304 @dimiro1 could you please make a PR out of it? http://thrift.apache.org/docs/HowToContribute --- If your project is set up for it, you can reply to this email and

[GitHub] thrift pull request: Thrift 3251 Add http transport for server to ...

2016-01-10 Thread dimiro1
Github user dimiro1 commented on the pull request: https://github.com/apache/thrift/pull/557#issuecomment-170362506 @Jens-G The current test infrastructure for the go lib explicitly assume that the server is a SocketServer /test/go/src/server.go. I think it will be a lot of work

[GitHub] thrift pull request: Thrift 3251 Add http transport for server to ...

2016-01-10 Thread Jens-G
Github user Jens-G commented on the pull request: https://github.com/apache/thrift/pull/557#issuecomment-170364590 All languages are encouraged 1) to implement the standard Thrift test. This test is the basic test to ensure interoperability across languages for all features

[GitHub] thrift pull request: Thrift 3251 Add http transport for server to ...

2016-01-09 Thread dimiro1
Github user dimiro1 commented on the pull request: https://github.com/apache/thrift/pull/557#issuecomment-170301847 I implemented with a handler func. https://gist.github.com/dimiro1/d38aaf4b7d84c2d5c8f0 --- If your project is set up for it, you can reply to this email and have your

[GitHub] thrift pull request: Thrift 3251 Add http transport for server to ...

2015-07-21 Thread hhkbp2
GitHub user hhkbp2 opened a pull request: https://github.com/apache/thrift/pull/557 Thrift 3251 Add http transport for server to Go lib Hi, This PR is a reopen for previous PR #556, in order to follow our how to contribute guide. It adds http transport for server