[GitHub] thrift pull request: THRIFT-3408: correct emitted JSON types

2015-11-03 Thread aughr
GitHub user aughr opened a pull request: https://github.com/apache/thrift/pull/679 THRIFT-3408: correct emitted JSON types https://issues.apache.org/jira/browse/THRIFT-3408 Currently this has no tests associated with it because the JSON generator itself appears untested.

[jira] [Created] (THRIFT-3408) JSON generator emits incorrect types

2015-11-03 Thread Andrew Bloomgarden (JIRA)
Andrew Bloomgarden created THRIFT-3408: -- Summary: JSON generator emits incorrect types Key: THRIFT-3408 URL: https://issues.apache.org/jira/browse/THRIFT-3408 Project: Thrift Issue

[GitHub] thrift pull request: THRIFT-338 Compact Protocol in Erlang

2015-11-03 Thread nsuke
Github user nsuke commented on the pull request: https://github.com/apache/thrift/pull/675#issuecomment-153348137 b31f0900b57b7473a8f84c7b74f73381dcff2818 --- 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

[GitHub] thrift pull request: THRIFT-338 Compact Protocol in Erlang

2015-11-03 Thread nsuke
Github user nsuke closed the pull request at: https://github.com/apache/thrift/pull/675 --- 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: THRIFT-3274 calling "make clean" twice in a r...

2015-11-03 Thread nsuke
GitHub user nsuke opened a pull request: https://github.com/apache/thrift/pull/676 THRIFT-3274 calling "make clean" twice in a row yields make error Removing `gen-cpp/.deps` hidden directory broke the source tree where any subsequent make command failed. You can merge this pull

[jira] [Commented] (THRIFT-3274) calling "make clean" twice in a row yields make error

2015-11-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3274?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14987252#comment-14987252 ] ASF GitHub Bot commented on THRIFT-3274: GitHub user nsuke opened a pull request:

[GitHub] thrift pull request: THRIFT-2423: Facebook's THeader protocol and ...

2015-11-03 Thread bufferoverflow
Github user bufferoverflow commented on the pull request: https://github.com/apache/thrift/pull/357#issuecomment-153325696 @nsuke Yes, please do so. --- 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

[jira] [Resolved] (THRIFT-338) Compact Protocol in Erlang

2015-11-03 Thread Aki Sukegawa (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-338?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aki Sukegawa resolved THRIFT-338. - Resolution: Fixed committed > Compact Protocol in Erlang > -- > >

[jira] [Commented] (THRIFT-338) Compact Protocol in Erlang

2015-11-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-338?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14987241#comment-14987241 ] ASF GitHub Bot commented on THRIFT-338: --- Github user nsuke closed the pull request at:

[jira] [Commented] (THRIFT-338) Compact Protocol in Erlang

2015-11-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-338?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14987240#comment-14987240 ] ASF GitHub Bot commented on THRIFT-338: --- Github user nsuke commented on the pull request:

[jira] [Commented] (THRIFT-2423) THeader

2015-11-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2423?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14987124#comment-14987124 ] ASF GitHub Bot commented on THRIFT-2423: Github user bufferoverflow commented on the pull

[jira] [Commented] (THRIFT-3408) JSON generator emits incorrect types

2015-11-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14987449#comment-14987449 ] ASF GitHub Bot commented on THRIFT-3408: GitHub user aughr opened a pull request:

[GitHub] thrift pull request: Thrift 1857 redux 2

2015-11-03 Thread nsuke
GitHub user nsuke opened a pull request: https://github.com/apache/thrift/pull/680 Thrift 1857 redux 2 Rational: * No TType.UTF8 (don't break cross-lang compatibility) * No six.BufferIO (don't break fastbinary) * Don't make py2 read/writeString UTF8 (don't break backward

[jira] [Assigned] (THRIFT-3200) JS and nodejs do not encode JSON protocol binary fields as base64

2015-11-03 Thread Aki Sukegawa (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3200?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aki Sukegawa reassigned THRIFT-3200: Assignee: Aki Sukegawa > JS and nodejs do not encode JSON protocol binary fields as

[GitHub] thrift pull request: Thrift-1857: Python 3 support, redux

2015-11-03 Thread nsuke
Github user nsuke commented on the pull request: https://github.com/apache/thrift/pull/213#issuecomment-153393441 I did some research on this. TType.UTF8 was added years ago but currently not used by any language. So using this now will result in incompatibility against any

[jira] [Assigned] (THRIFT-3350) Python JSON protocol does not encode binary as Base64

2015-11-03 Thread Aki Sukegawa (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3350?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aki Sukegawa reassigned THRIFT-3350: Assignee: Aki Sukegawa > Python JSON protocol does not encode binary as Base64 >

[jira] [Commented] (THRIFT-2423) THeader

2015-11-03 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2423?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14987606#comment-14987606 ] Hudson commented on THRIFT-2423: SUCCESS: Integrated in Thrift #1708 (See

[GitHub] thrift pull request: THRIFT-3182: Close transport after reading an...

2015-11-03 Thread marshalium
Github user marshalium commented on the pull request: https://github.com/apache/thrift/pull/512#issuecomment-153496623 Hey @jfarrell. Are things better with CI now? Should I rebase off of master and try again? Thanks! --- If your project is set up for it, you can reply to this email

[jira] [Commented] (THRIFT-3182) TFramedTransport is in an invalid state after frame size exception

2015-11-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3182?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14988175#comment-14988175 ] ASF GitHub Bot commented on THRIFT-3182: Github user marshalium commented on the pull request:

[GitHub] thrift pull request: Allow context to be passed into NewTHttpPostC...

2015-11-03 Thread i
GitHub user i opened a pull request: https://github.com/apache/thrift/pull/682 Allow context to be passed into NewTHttpPostClient This provides a way to cancel outgoing requests which this library doesn't currently support. You can merge this pull request into a Git repository

[jira] [Resolved] (THRIFT-3274) calling "make clean" twice in a row yields make error

2015-11-03 Thread Roger Meier (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3274?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Roger Meier resolved THRIFT-3274. - Resolution: Fixed tutorial fix is committed > calling "make clean" twice in a row yields make

[jira] [Commented] (THRIFT-3274) calling "make clean" twice in a row yields make error

2015-11-03 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3274?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14988407#comment-14988407 ] Hudson commented on THRIFT-3274: SUCCESS: Integrated in Thrift #1710 (See

[jira] [Commented] (THRIFT-3274) calling "make clean" twice in a row yields make error

2015-11-03 Thread Jens Geyer (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3274?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14988017#comment-14988017 ] Jens Geyer commented on THRIFT-3274: Better, but not perfect. The tutorial seems to suffer from the

[jira] [Comment Edited] (THRIFT-3274) calling "make clean" twice in a row yields make error

2015-11-03 Thread Jens Geyer (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3274?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14988017#comment-14988017 ] Jens Geyer edited comment on THRIFT-3274 at 11/3/15 8:09 PM: - Better, but not

[jira] [Resolved] (THRIFT-3274) calling "make clean" twice in a row yields make error

2015-11-03 Thread Roger Meier (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3274?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Roger Meier resolved THRIFT-3274. - Resolution: Fixed Assignee: Aki Sukegawa Thanks Aki! > calling "make clean" twice in a

[jira] [Reopened] (THRIFT-3274) calling "make clean" twice in a row yields make error

2015-11-03 Thread Jens Geyer (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3274?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jens Geyer reopened THRIFT-3274: > calling "make clean" twice in a row yields make error >

[jira] [Commented] (THRIFT-3274) calling "make clean" twice in a row yields make error

2015-11-03 Thread Roger Meier (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3274?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14988135#comment-14988135 ] Roger Meier commented on THRIFT-3274: - [~jensg] fix for tutorial is in verification... > calling

[GitHub] thrift pull request: THRIFT-3274 calling "make clean" twice in a r...

2015-11-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/thrift/pull/676 --- 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-3274) calling "make clean" twice in a row yields make error

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

[jira] [Commented] (THRIFT-3409) NodeJS binary field issues

2015-11-03 Thread Roger Meier (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3409?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14988006#comment-14988006 ] Roger Meier commented on THRIFT-3409: - great stuff! +1 > NodeJS binary field issues >

[jira] [Commented] (THRIFT-3407) Refill buffer when JSON closing bracket is missing

2015-11-03 Thread Roger Meier (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3407?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14987994#comment-14987994 ] Roger Meier commented on THRIFT-3407: - +1 > Refill buffer when JSON closing bracket is missing >

[jira] [Reopened] (THRIFT-3339) Support for database/sql

2015-11-03 Thread Adam Beberg (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3339?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Adam Beberg reopened THRIFT-3339: - Reopening to add support for optional enums. > Support for database/sql >

[GitHub] thrift pull request: Deprecate this Apache Thrift Fork

2015-11-03 Thread Raynos
Github user Raynos commented on the pull request: https://github.com/apache/thrift/pull/683#issuecomment-153534427 #legit --- 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

[GitHub] thrift pull request: Deprecate this Apache Thrift Fork

2015-11-03 Thread breerly
GitHub user breerly opened a pull request: https://github.com/apache/thrift/pull/683 Deprecate this Apache Thrift Fork We're finally to a place where we can deprecate this repo. * Python & Node use [thriftrw](https://github.com/search?utf8=%E2%9C%93=thriftrw). * Go

[GitHub] thrift pull request: Thrift 1857 redux 2

2015-11-03 Thread bufferoverflow
Github user bufferoverflow commented on the pull request: https://github.com/apache/thrift/pull/680#issuecomment-153534017 :+1: tested with autoconf running **make cross** and cmake **make** on Debian Jessie amd64 with Python 2.7.9 -roger --- If your project is set up for

[GitHub] thrift pull request: Deprecate this Apache Thrift Fork

2015-11-03 Thread breerly
Github user breerly closed the pull request at: https://github.com/apache/thrift/pull/683 --- 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: Deprecate this Apache Thrift Fork

2015-11-03 Thread kriskowal
Github user kriskowal commented on the pull request: https://github.com/apache/thrift/pull/683#issuecomment-153534454 :ok: --- 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

[GitHub] thrift pull request: Deprecate this Apache Thrift Fork

2015-11-03 Thread Raynos
Github user Raynos commented on the pull request: https://github.com/apache/thrift/pull/683#issuecomment-153534582 #oops --- 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

[GitHub] thrift pull request: Fix package.json to include only the needed f...

2015-11-03 Thread bufferoverflow
Github user bufferoverflow commented on the pull request: https://github.com/apache/thrift/pull/672#issuecomment-153511243 Thanks for the PR! We need a strong YES to according to the contribution guidelines as mentioned here https://thrift.apache.org/docs/HowToContribute

[GitHub] thrift pull request: THRIFT-3407 Refill buffer when JSON closing b...

2015-11-03 Thread nsuke
GitHub user nsuke opened a pull request: https://github.com/apache/thrift/pull/678 THRIFT-3407 Refill buffer when JSON closing bracket is missing You can merge this pull request into a Git repository by running: $ git pull https://github.com/nsuke/thrift THRIFT-3407

[jira] [Commented] (THRIFT-3407) Refill buffer when JSON closing bracket is missing

2015-11-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3407?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14987399#comment-14987399 ] ASF GitHub Bot commented on THRIFT-3407: GitHub user nsuke opened a pull request:

[GitHub] thrift pull request: THRIFT-3409 NodeJS binary field issues

2015-11-03 Thread nsuke
GitHub user nsuke opened a pull request: https://github.com/apache/thrift/pull/681 THRIFT-3409 NodeJS binary field issues You can merge this pull request into a Git repository by running: $ git pull https://github.com/nsuke/thrift THRIFT-3409 Alternatively you can review and

[jira] [Closed] (THRIFT-338) Compact Protocol in Erlang

2015-11-03 Thread Aki Sukegawa (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-338?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aki Sukegawa closed THRIFT-338. --- > Compact Protocol in Erlang > -- > > Key: THRIFT-338 >

[jira] [Created] (THRIFT-3409) NodeJS binary field issues

2015-11-03 Thread Aki Sukegawa (JIRA)
Aki Sukegawa created THRIFT-3409: Summary: NodeJS binary field issues Key: THRIFT-3409 URL: https://issues.apache.org/jira/browse/THRIFT-3409 Project: Thrift Issue Type: Sub-task

[jira] [Commented] (THRIFT-3409) NodeJS binary field issues

2015-11-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3409?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14987525#comment-14987525 ] ASF GitHub Bot commented on THRIFT-3409: GitHub user nsuke opened a pull request:

[jira] [Commented] (THRIFT-2423) THeader

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

[jira] [Resolved] (THRIFT-2423) THeader

2015-11-03 Thread Aki Sukegawa (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2423?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aki Sukegawa resolved THRIFT-2423. -- Resolution: Fixed Assignee: Dave Watson Committed, sorry for the delay. Thanks ! >

[GitHub] thrift pull request: THRIFT-2423: Facebook's THeader protocol and ...

2015-11-03 Thread nsuke
Github user nsuke commented on the pull request: https://github.com/apache/thrift/pull/357#issuecomment-153356845 There were several build issues. I'll commit the fixed version shortly after #677. --- If your project is set up for it, you can reply to this email and have your

[jira] [Commented] (THRIFT-338) Compact Protocol in Erlang

2015-11-03 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-338?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14987298#comment-14987298 ] Hudson commented on THRIFT-338: --- SUCCESS: Integrated in Thrift #1707 (See

[jira] [Commented] (THRIFT-2423) THeader

2015-11-03 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2423?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14987297#comment-14987297 ] ASF GitHub Bot commented on THRIFT-2423: Github user nsuke commented on the pull request:

[jira] [Created] (THRIFT-3407) Refill buffer when JSON closing bracket is missing

2015-11-03 Thread Aki Sukegawa (JIRA)
Aki Sukegawa created THRIFT-3407: Summary: Refill buffer when JSON closing bracket is missing Key: THRIFT-3407 URL: https://issues.apache.org/jira/browse/THRIFT-3407 Project: Thrift Issue

[jira] [Updated] (THRIFT-3407) Refill buffer when JSON closing bracket is missing

2015-11-03 Thread Aki Sukegawa (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3407?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aki Sukegawa updated THRIFT-3407: - Description: NodeJS TJSONProtocol aborts reading when number of closing brackets is smaller

[jira] [Updated] (THRIFT-3407) Refill buffer when JSON closing bracket is missing

2015-11-03 Thread Aki Sukegawa (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3407?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aki Sukegawa updated THRIFT-3407: - Description: NodeJS TJSONProtocol aborts reading when number of closing brackets is smaller

[GitHub] thrift pull request: Thrift 2423 redux

2015-11-03 Thread nsuke
GitHub user nsuke opened a pull request: https://github.com/apache/thrift/pull/677 Thrift 2423 redux You can merge this pull request into a Git repository by running: $ git pull https://github.com/nsuke/thrift THRIFT-2423-redux Alternatively you can review and apply these

[GitHub] thrift pull request: THRIFT-2423: Facebook's THeader protocol and ...

2015-11-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/thrift/pull/357 --- 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: Thrift 2423 redux

2015-11-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/thrift/pull/677 --- 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