[GitHub] thrift pull request #1297: handleRuntimeExceptions

2017-06-26 Thread zhouxianjun
Github user zhouxianjun closed the pull request at: https://github.com/apache/thrift/pull/1297 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] thrift issue #1297: handleRuntimeExceptions

2017-06-26 Thread zhouxianjun
Github user zhouxianjun commented on the issue: https://github.com/apache/thrift/pull/1297 I was wrong, I compiled the master, but I used 0.10.0, I'm sorry. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[jira] [Updated] (THRIFT-4237) Go TServerSocket Race Conditions

2017-06-26 Thread Zach Wasserman (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4237?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Zach Wasserman updated THRIFT-4237: --- Description: Run the following test with {{go test -race -run TestSocketConcurrency}}:

[jira] [Created] (THRIFT-4237) Go TServerSocket Race Conditions

2017-06-26 Thread Zach Wasserman (JIRA)
Zach Wasserman created THRIFT-4237: -- Summary: Go TServerSocket Race Conditions Key: THRIFT-4237 URL: https://issues.apache.org/jira/browse/THRIFT-4237 Project: Thrift Issue Type: Bug

[GitHub] thrift issue #1297: handleRuntimeExceptions

2017-06-26 Thread Jens-G
Github user Jens-G commented on the issue: https://github.com/apache/thrift/pull/1297 Something is wrong here. Could you please review your pull request? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[jira] [Resolved] (THRIFT-4234) Travis build fails cross language tests with "Unsupported security protocol type"

2017-06-26 Thread Jens Geyer (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4234?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jens Geyer resolved THRIFT-4234. Resolution: Fixed Fix Version/s: 0.11.0 Committed, thanks! > Travis build fails cross

[jira] [Commented] (THRIFT-4234) Travis build fails cross language tests with "Unsupported security protocol type"

2017-06-26 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4234?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16063815#comment-16063815 ] ASF GitHub Bot commented on THRIFT-4234: Github user asfgit closed the pull request at:

[jira] [Commented] (THRIFT-4213) Travis build fails at curl -sSL https://www.npmjs.com/install.sh | sh

2017-06-26 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4213?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16063814#comment-16063814 ] ASF GitHub Bot commented on THRIFT-4213: Github user asfgit closed the pull request at:

[jira] [Commented] (THRIFT-4213) Travis build fails at curl -sSL https://www.npmjs.com/install.sh | sh

2017-06-26 Thread Jens Geyer (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4213?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16063817#comment-16063817 ] Jens Geyer commented on THRIFT-4213: Good question. Since tzhe overall result has improved, I just

[GitHub] thrift pull request #1296: THRIFT-4234 Travis build fails cross language tes...

2017-06-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/thrift/pull/1296 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] thrift pull request #1295: THRIFT-4213 Travis build fails at curl -sSL https...

2017-06-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/thrift/pull/1295 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[jira] [Commented] (THRIFT-4236) Support context in go generated code.

2017-06-26 Thread Jens Geyer (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16063806#comment-16063806 ] Jens Geyer commented on THRIFT-4236: {quote} > other framework like grpc will pass a context for the

[jira] [Commented] (THRIFT-4234) Travis build fails cross language tests with "Unsupported security protocol type"

2017-06-26 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4234?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16063734#comment-16063734 ] ASF GitHub Bot commented on THRIFT-4234: Github user Jens-G commented on the issue:

[GitHub] thrift issue #1296: THRIFT-4234 Travis build fails cross language tests with...

2017-06-26 Thread Jens-G
Github user Jens-G commented on the issue: https://github.com/apache/thrift/pull/1296 Nice, but not 100%. Still two are saying .. > npm install > /bin/bash: npm: command not found > --- If your project is set up for it, you can reply to this email and have your

Re: Broken Build?

2017-06-26 Thread Eric Conner
Ok, these 2 PRs together should fix the build: https://github.com/apache/thrift/pull/1296 https://github.com/apache/thrift/pull/1295 On Sat, Jun 24, 2017 at 7:04 PM, Eric Conner wrote: > Great, I've updated the issue with a PR and some comments. We can move > the

[jira] [Commented] (THRIFT-4236) Support context in go generated code.

2017-06-26 Thread taozle (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16062841#comment-16062841 ] taozle commented on THRIFT-4236: I don't think the {{THeaderTransport}} is what i want. The {{context}}

[jira] [Commented] (THRIFT-4236) Support context in go generated code.

2017-06-26 Thread Jens Geyer (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16062790#comment-16062790 ] Jens Geyer commented on THRIFT-4236: There is {{THeaderTransport}}, currently only implemented for

[GitHub] thrift pull request #1297: handleRuntimeExceptions

2017-06-26 Thread zhouxianjun
GitHub user zhouxianjun opened a pull request: https://github.com/apache/thrift/pull/1297 handleRuntimeExceptions RT: thrift command: 0.10.0 thrift java lib: 0.10.0 Method does not override method from its superclass org.apache.thrift.ProcessFunction is no