[jira] [Created] (THRIFT-3521) TSimpleJSONProtocolTest is not deterministic (fails when run on JDK 8)

2016-01-05 Thread Benjamin Gould (JIRA)
Benjamin Gould created THRIFT-3521: -- Summary: TSimpleJSONProtocolTest is not deterministic (fails when run on JDK 8) Key: THRIFT-3521 URL: https://issues.apache.org/jira/browse/THRIFT-3521 Project:

[jira] [Commented] (THRIFT-3521) TSimpleJSONProtocolTest is not deterministic (fails when run on JDK 8)

2016-01-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15083412#comment-15083412 ] ASF GitHub Bot commented on THRIFT-3521: GitHub user bgould opened a pull request:

[GitHub] thrift pull request: THRIFT-3521 : TSimpleJSONProtocolTest is not ...

2016-01-05 Thread bgould
GitHub user bgould opened a pull request: https://github.com/apache/thrift/pull/772 THRIFT-3521 : TSimpleJSONProtocolTest is not deterministic (fails when run on JDK 8) This PR removes fields that cause TSimpleJSONProtocolTest to fail when run using different version of the Java

[jira] [Commented] (THRIFT-3521) TSimpleJSONProtocolTest is not deterministic (fails when run on JDK 8)

2016-01-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15083620#comment-15083620 ] ASF GitHub Bot commented on THRIFT-3521: Github user nsuke commented on the pull request:

[jira] [Resolved] (THRIFT-3519) fastbinary does not work with -gen py:utf8strings

2016-01-05 Thread Aki Sukegawa (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3519?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aki Sukegawa resolved THRIFT-3519. -- Resolution: Fixed Fix Version/s: 1.0 > fastbinary does not work with -gen

[jira] [Created] (THRIFT-3522) In multiplexer mode, when data gets big, error occurs in accepting data

2016-01-05 Thread Amos Chen (JIRA)
Amos Chen created THRIFT-3522: - Summary: In multiplexer mode, when data gets big, error occurs in accepting data Key: THRIFT-3522 URL: https://issues.apache.org/jira/browse/THRIFT-3522 Project: Thrift

[GitHub] thrift pull request: THRIFT-3521 : TSimpleJSONProtocolTest is not ...

2016-01-05 Thread nsuke
Github user nsuke commented on the pull request: https://github.com/apache/thrift/pull/772#issuecomment-169105689 I was mainly concerned about completely losing map/set tests. Of course preserving all tests for those element types is great, but I think it's fine to unset most of

[GitHub] thrift pull request: THRIFT-3521 : TSimpleJSONProtocolTest is not ...

2016-01-05 Thread nsuke
Github user nsuke commented on the pull request: https://github.com/apache/thrift/pull/772#issuecomment-169092979 @bgould, thanks for looking into this. Would it be tedious and/or complicate the code if we match against regex instead of removing them ? --- If your project is set

[jira] [Commented] (THRIFT-3521) TSimpleJSONProtocolTest is not deterministic (fails when run on JDK 8)

2016-01-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15083540#comment-15083540 ] ASF GitHub Bot commented on THRIFT-3521: Github user nsuke commented on the pull request:

[GitHub] thrift pull request: THRIFT-3521 : TSimpleJSONProtocolTest is not ...

2016-01-05 Thread bgould
Github user bgould commented on the pull request: https://github.com/apache/thrift/pull/772#issuecomment-169096832 I thought about that too but didn't have time to do it this morning and just wanted to get a working build. I think that perhaps better than a regex might be to

[jira] [Commented] (THRIFT-3521) TSimpleJSONProtocolTest is not deterministic (fails when run on JDK 8)

2016-01-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15083559#comment-15083559 ] ASF GitHub Bot commented on THRIFT-3521: Github user bgould commented on the pull request:

[jira] [Commented] (THRIFT-2994) Node.js TJSONProtocol cannot be used for object serialization.

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

[jira] [Resolved] (THRIFT-2994) Node.js TJSONProtocol cannot be used for object serialization.

2016-01-05 Thread Aki Sukegawa (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2994?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aki Sukegawa resolved THRIFT-2994. -- Resolution: Fixed Assignee: Jan Brauer (was: Randy Abernethy) Fix Version/s:

[GitHub] thrift pull request: THRIFT-9999:In multiplexer mode, when data ge...

2016-01-05 Thread mice530
GitHub user mice530 opened a pull request: https://github.com/apache/thrift/pull/773 THRIFT-:In multiplexer mode, when data gets big, error occurs in accepting data THRIFT-:In multiplexer mode, when data gets big, delete hash of seqId may lead to wrong client. You can

[GitHub] thrift pull request: THRIFT-3519 fastbinary does not work with -ge...

2016-01-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/thrift/pull/769 --- 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-3519) fastbinary does not work with -gen py:utf8strings

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

[jira] [Commented] (THRIFT-3437) Mingw-w64 build fail

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

[GitHub] thrift pull request: THRIFT-2994: Fix TJsonProtocol for nodejs

2016-01-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/thrift/pull/379 --- 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] [Resolved] (THRIFT-3437) Mingw-w64 build fail

2016-01-05 Thread Aki Sukegawa (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3437?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aki Sukegawa resolved THRIFT-3437. -- Resolution: Fixed Fix Version/s: 1.0 Committed. The patch presumably resolves 2 other

[jira] [Commented] (THRIFT-3521) TSimpleJSONProtocolTest is not deterministic (fails when run on JDK 8)

2016-01-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15084585#comment-15084585 ] ASF GitHub Bot commented on THRIFT-3521: Github user nsuke commented on the pull request:

[GitHub] thrift pull request: THRIFT-3521 : TSimpleJSONProtocolTest is not ...

2016-01-05 Thread nsuke
Github user nsuke commented on the pull request: https://github.com/apache/thrift/pull/772#issuecomment-169204403 You're right, you were already keeping those which still work for JDK8. --- If your project is set up for it, you can reply to this email and have your reply appear on

[jira] [Commented] (THRIFT-2427) Add support for Multiplexed Async Processors

2016-01-05 Thread Nikita Dolgov (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2427?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15084953#comment-15084953 ] Nikita Dolgov commented on THRIFT-2427: --- In case it might inspire someone to do a code review, I'd

[jira] [Resolved] (THRIFT-3521) TSimpleJSONProtocolTest is not deterministic (fails when run on JDK 8)

2016-01-05 Thread Aki Sukegawa (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aki Sukegawa resolved THRIFT-3521. -- Resolution: Fixed committed, thanks > TSimpleJSONProtocolTest is not deterministic (fails

[jira] [Commented] (THRIFT-3521) TSimpleJSONProtocolTest is not deterministic (fails when run on JDK 8)

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

[GitHub] thrift pull request: THRIFT-3521 : TSimpleJSONProtocolTest is not ...

2016-01-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/thrift/pull/772 --- 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] [Updated] (THRIFT-3523) XML Generator

2016-01-05 Thread Benjamin Gould (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3523?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Benjamin Gould updated THRIFT-3523: --- Description: I've put together an XML model for Thrift IDL that is very similar in concept

[jira] [Created] (THRIFT-3523) XML Generator

2016-01-05 Thread Benjamin Gould (JIRA)
Benjamin Gould created THRIFT-3523: -- Summary: XML Generator Key: THRIFT-3523 URL: https://issues.apache.org/jira/browse/THRIFT-3523 Project: Thrift Issue Type: New Feature

[GitHub] thrift pull request: THRIFT-3523 : XML Generator

2016-01-05 Thread bgould
GitHub user bgould opened a pull request: https://github.com/apache/thrift/pull/774 THRIFT-3523 : XML Generator I've put together an XML model for Thrift IDL that is very similar in concept and structure to the model created by the JSON generator of the Thrift compiler. The code

[jira] [Commented] (THRIFT-3523) XML Generator

2016-01-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3523?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15084574#comment-15084574 ] ASF GitHub Bot commented on THRIFT-3523: GitHub user bgould opened a pull request:

[GitHub] thrift pull request: THRIFT-9999:In multiplexer mode, when data ge...

2016-01-05 Thread Jens-G
Github user Jens-G commented on the pull request: https://github.com/apache/thrift/pull/773#issuecomment-169119235 Could you please follow our small set of [contribution guidelines](http://thrift.apache.org/docs/HowToContribute) and file a valid JIRA ticket for it, ideally including

[jira] [Commented] (THRIFT-3466) Set Parameters Have Wrong Key Type in Go Code

2016-01-05 Thread Tom Deering (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3466?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15083997#comment-15083997 ] Tom Deering commented on THRIFT-3466: - This appears to affect lists as well as sets. > Set

[jira] [Commented] (THRIFT-3514) PHP 7 compatible version of binary protocol

2016-01-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3514?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15083921#comment-15083921 ] ASF GitHub Bot commented on THRIFT-3514: Github user dsp commented on the pull request:

[GitHub] thrift pull request: THRIFT-3514: Add PHP 7 version of php_thrift_...

2016-01-05 Thread dsp
Github user dsp commented on the pull request: https://github.com/apache/thrift/pull/765#issuecomment-169147175 The contbuild error seem to be in java-csharp and perl-py3 which are unrelated. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] thrift pull request: THRIFT-3521 : TSimpleJSONProtocolTest is not ...

2016-01-05 Thread bgould
Github user bgould commented on the pull request: https://github.com/apache/thrift/pull/772#issuecomment-169139570 I see... in that I think that the tests should be ok as they are because there still exist maps and sets in the `testSanePartsOfCompactProtoTestStruct` test method.

[jira] [Commented] (THRIFT-3521) TSimpleJSONProtocolTest is not deterministic (fails when run on JDK 8)

2016-01-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3521?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15083838#comment-15083838 ] ASF GitHub Bot commented on THRIFT-3521: Github user bgould commented on the pull request:

[jira] [Commented] (THRIFT-3514) PHP 7 compatible version of binary protocol

2016-01-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3514?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15082972#comment-15082972 ] ASF GitHub Bot commented on THRIFT-3514: Github user ovr commented on the pull request:

[GitHub] thrift pull request: THRIFT-3514: Add PHP 7 version of php_thrift_...

2016-01-05 Thread ovr
Github user ovr commented on the pull request: https://github.com/apache/thrift/pull/765#issuecomment-168986427 Awesome!! :cake: :beers: --- 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

[jira] [Commented] (THRIFT-3474) Docker: thrift-compiler

2016-01-05 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3474?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15084531#comment-15084531 ] ASF GitHub Bot commented on THRIFT-3474: Github user jfarrell commented on the pull request:

[GitHub] thrift pull request: THRIFT-3474 Docker: thrift-compiler

2016-01-05 Thread jfarrell
Github user jfarrell commented on the pull request: https://github.com/apache/thrift/pull/733#issuecomment-169193943 @nsuke correct, gofmt is called in the compiler --- 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

[jira] [Updated] (THRIFT-2426) clarify IP rights and contributions from fbthrift

2016-01-05 Thread Jake Farrell (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2426?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jake Farrell updated THRIFT-2426: - Fix Version/s: (was: 0.9.2) 1.0 > clarify IP rights and contributions

[jira] [Updated] (THRIFT-3449) TBaseAsyncProcessor fb.responseReady() never called for oneway functions

2016-01-05 Thread Jake Farrell (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3449?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jake Farrell updated THRIFT-3449: - Fix Version/s: (was: 0.9.3) 0.9.4 > TBaseAsyncProcessor

[jira] [Closed] (THRIFT-3518) TConcurrentClientSyncInfo files were missing for Visual Studio

2016-01-05 Thread Jake Farrell (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3518?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jake Farrell closed THRIFT-3518. Resolution: Fixed committed, thanks for the patch > TConcurrentClientSyncInfo files were missing

[jira] [Updated] (THRIFT-3517) Default value of enum is starting at zero in Java

2016-01-05 Thread Jake Farrell (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3517?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jake Farrell updated THRIFT-3517: - Component/s: Tutorial > Default value of enum is starting at zero in Java >

[jira] [Closed] (THRIFT-1007) Deserializer override for ByteBuffer input

2016-01-05 Thread Jake Farrell (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-1007?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jake Farrell closed THRIFT-1007. Resolution: Won't Fix Assignee: Jake Farrell closing as this has been around since 0.5 and

[jira] [Closed] (THRIFT-449) The wire format of the JSON Protocol may not always be valid JSON if it contains non-UTF8 encoded strings

2016-01-05 Thread Jake Farrell (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-449?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jake Farrell closed THRIFT-449. --- Resolution: Won't Fix Assignee: Jake Farrell Fix Version/s: 0.9.4 closing as this has