[jira] [Commented] (THRIFT-4480) NodeJS warning on binary_protocol writeMessageEnd when seqid = 0

2018-01-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16346303#comment-16346303 ] ASF GitHub Bot commented on THRIFT-4480: GitHub user bforbis opened a pull request:

[GitHub] thrift pull request #1487: THRIFT-4480 - Handle seqid = 0

2018-01-30 Thread bforbis
GitHub user bforbis opened a pull request: https://github.com/apache/thrift/pull/1487 THRIFT-4480 - Handle seqid = 0 You can merge this pull request into a Git repository by running: $ git pull https://github.com/bforbis/thrift THRIFT-4480 Alternatively you can review and

[jira] [Commented] (THRIFT-4484) C++ codegen invalid for optional self-membership

2018-01-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16346264#comment-16346264 ] ASF GitHub Bot commented on THRIFT-4484: Github user ringerc commented on the issue:

[jira] [Updated] (THRIFT-4484) C++ codegen invalid for optional self-membership

2018-01-30 Thread Craig Ringer (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4484?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Craig Ringer updated THRIFT-4484: - Description: Support was added for self-referential objects in in

[GitHub] thrift issue #84: Tree/Recursive struct support in thrift

2018-01-30 Thread ringerc
Github user ringerc commented on the issue: https://github.com/apache/thrift/pull/84 Created as https://issues.apache.org/jira/browse/THRIFT-4484 ---

[jira] [Created] (THRIFT-4484) C++ codegen invalid for optional self-membership

2018-01-30 Thread Craig Ringer (JIRA)
Craig Ringer created THRIFT-4484: Summary: C++ codegen invalid for optional self-membership Key: THRIFT-4484 URL: https://issues.apache.org/jira/browse/THRIFT-4484 Project: Thrift Issue

[GitHub] thrift issue #84: Tree/Recursive struct support in thrift

2018-01-30 Thread ringerc
Github user ringerc commented on the issue: https://github.com/apache/thrift/pull/84 I see a bunch of mutual recursive structure support in the tests, but nothing for an optional single self-referential member, like ``` struct RecSelf { 1: i16 item 2:

[jira] [Created] (THRIFT-4483) Java TSaslTransport does not respect SASL RAW_SEND_SIZE

2018-01-30 Thread Dan Burkert (JIRA)
Dan Burkert created THRIFT-4483: --- Summary: Java TSaslTransport does not respect SASL RAW_SEND_SIZE Key: THRIFT-4483 URL: https://issues.apache.org/jira/browse/THRIFT-4483 Project: Thrift Issue

[GitHub] thrift issue #1380: Support x64 build mode on Windows with DMD v2.076.0.

2018-01-30 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1380 I can't (well, won't) merge a PR with 100 commits. I also fixed dmd invocation on linux earlier today and fixed the link ordering issue. So if you can rebase and squash to one commit, that would

[jira] [Created] (THRIFT-4482) error making c++ library on windows 8.1 using cygwin

2018-01-30 Thread Michel Max (JIRA)
Michel Max created THRIFT-4482: -- Summary: error making c++ library on windows 8.1 using cygwin Key: THRIFT-4482 URL: https://issues.apache.org/jira/browse/THRIFT-4482 Project: Thrift Issue

[jira] [Created] (THRIFT-4481) TBinaryProtocol.writeMessageEnd isn't trowable exception

2018-01-30 Thread Anton Shchyrov (JIRA)
Anton Shchyrov created THRIFT-4481: -- Summary: TBinaryProtocol.writeMessageEnd isn't trowable exception Key: THRIFT-4481 URL: https://issues.apache.org/jira/browse/THRIFT-4481 Project: Thrift

[jira] [Commented] (THRIFT-4337) Able to set keyStore and trustStore as InputStream in the TSSLTransportFactory.TSSLTransportParameters

2018-01-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16345284#comment-16345284 ] ASF GitHub Bot commented on THRIFT-4337: GitHub user dmvolod opened a pull request:

[GitHub] thrift pull request #1486: THRIFT-4337: Able to set keyStore and trustStore ...

2018-01-30 Thread dmvolod
GitHub user dmvolod opened a pull request: https://github.com/apache/thrift/pull/1486 THRIFT-4337: Able to set keyStore and trustStore as InputStream in the TSSLTransportFactory.TSSLTransportParameters Plus some tests cleanup You can merge this pull request into a Git repository

[jira] [Updated] (THRIFT-4480) NodeJS warning on binary_protocol writeMessageEnd when seqid = 0

2018-01-30 Thread Brian Forbis (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4480?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Brian Forbis updated THRIFT-4480: - Priority: Minor (was: Trivial) > NodeJS warning on binary_protocol writeMessageEnd when seqid =

[jira] [Created] (THRIFT-4480) NodeJS warning on binary_protocol writeMessageEnd when seqid = 0

2018-01-30 Thread Brian Forbis (JIRA)
Brian Forbis created THRIFT-4480: Summary: NodeJS warning on binary_protocol writeMessageEnd when seqid = 0 Key: THRIFT-4480 URL: https://issues.apache.org/jira/browse/THRIFT-4480 Project: Thrift

[jira] [Commented] (THRIFT-4474) PHP generator use PSR-4 default

2018-01-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4474?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16345134#comment-16345134 ] ASF GitHub Bot commented on THRIFT-4474: Github user jeking3 commented on the issue:

[GitHub] thrift issue #1479: THRIFT-4474: generate PHP code use PSR-4 style default

2018-01-30 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1479 If we are making breaking changes, they need to be documented very clearly in the php/README.md file as part of any commit. See my breaking changes for 0.10.0 and 0.11.0 in the perl/README.md for

[jira] [Commented] (THRIFT-4474) PHP generator use PSR-4 default

2018-01-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4474?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16345043#comment-16345043 ] ASF GitHub Bot commented on THRIFT-4474: Github user RobberPhex commented on the issue:

[GitHub] thrift issue #1479: THRIFT-4474: generate PHP code use PSR-4 style default

2018-01-30 Thread RobberPhex
Github user RobberPhex commented on the issue: https://github.com/apache/thrift/pull/1479 Personally, I think we can mark classmap/old-style PHP compiler deprecated. And can be removed in future. And, ThriftClassLoader can also be removed in future, to migrate composer

[jira] [Commented] (THRIFT-4474) PHP generator use PSR-4 default

2018-01-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4474?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16345039#comment-16345039 ] ASF GitHub Bot commented on THRIFT-4474: Github user RobberPhex commented on the issue:

[GitHub] thrift issue #1479: THRIFT-4474: generate PHP code use PSR-4 style default

2018-01-30 Thread RobberPhex
Github user RobberPhex commented on the issue: https://github.com/apache/thrift/pull/1479 There are some break changes: 1. for generated code, old struct is `Types.php` for all args, results, `.php` for `ServiceIf`, `ServiceClient`, etc. new struct is `.php` for ``.

[GitHub] thrift issue #1484: TBufferedTransport must have underlying transport

2018-01-30 Thread RobberPhex
Github user RobberPhex commented on the issue: https://github.com/apache/thrift/pull/1484 CI isn't stable. It's so headache. :( ---

[jira] [Resolved] (THRIFT-4308) D language docker images need demios for libevent and openssl fixed to re-enable make cross on dlang

2018-01-30 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4308?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III resolved THRIFT-4308. Resolution: Fixed Fix Version/s: 0.12.0 > D language docker images need

[jira] [Commented] (THRIFT-4308) D language docker images need demios for libevent and openssl fixed to re-enable make cross on dlang

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

[GitHub] thrift pull request #1483: THRIFT-4308: re-enable dlang deimos build support...

2018-01-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/thrift/pull/1483 ---

[GitHub] thrift issue #1483: THRIFT-4308: re-enable dlang deimos build support for li...

2018-01-30 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1483 The concurrency test failed, which is a known intermittent issue. ---

[jira] [Commented] (THRIFT-4308) D language docker images need demios for libevent and openssl fixed to re-enable make cross on dlang

2018-01-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4308?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16344992#comment-16344992 ] ASF GitHub Bot commented on THRIFT-4308: Github user jeking3 commented on the issue:

[jira] [Updated] (THRIFT-4477) TBufferedTransport must have underlying transport

2018-01-30 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4477?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III updated THRIFT-4477: --- Priority: Minor (was: Major) > TBufferedTransport must have underlying transport >

[jira] [Commented] (THRIFT-4477) TBufferedTransport must have underlying transport

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

[GitHub] thrift pull request #1484: TBufferedTransport must have underlying transport

2018-01-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/thrift/pull/1484 ---

[jira] [Resolved] (THRIFT-4477) TBufferedTransport must have underlying transport

2018-01-30 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4477?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III resolved THRIFT-4477. Resolution: Fixed Fix Version/s: 0.12.0 Committed - thanks. >

[GitHub] thrift issue #1484: TBufferedTransport must have underlying transport

2018-01-30 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1484 The build failures are unrelated. mingw on Windows has been failing; go-dart is a new possibly intermittant cross test failure. ---

[GitHub] thrift pull request #1484: TBufferedTransport must have underlying transport

2018-01-30 Thread jeking3
Github user jeking3 commented on a diff in the pull request: https://github.com/apache/thrift/pull/1484#discussion_r164729770 --- Diff: lib/php/lib/Transport/TBufferedTransport.php --- @@ -22,6 +22,7 @@ namespace Thrift\Transport; +use

[GitHub] thrift pull request #1484: TBufferedTransport must have underlying transport

2018-01-30 Thread jeking3
Github user jeking3 commented on a diff in the pull request: https://github.com/apache/thrift/pull/1484#discussion_r164729591 --- Diff: lib/php/lib/Factory/TStringFuncFactory.php --- @@ -21,8 +21,9 @@ namespace Thrift\Factory; -use Thrift\StringFunc\Mbstring;

[jira] [Commented] (THRIFT-4429) Make TThreadPoolServer.executorService_ available in inherited classes and refactor methods to be able customization

2018-01-30 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16344718#comment-16344718 ] ASF GitHub Bot commented on THRIFT-4429: GitHub user dmvolod opened a pull request:

[GitHub] thrift pull request #1485: THRIFT-4429: Make TThreadPoolServer.executorServi...

2018-01-30 Thread dmvolod
GitHub user dmvolod opened a pull request: https://github.com/apache/thrift/pull/1485 THRIFT-4429: Make TThreadPoolServer.executorService_ available in inherited classes and refactor methods to be able customization Plus some classes cleanup from unused imports You can merge this