[jira] [Commented] (THRIFT-4326) Ruby BufferedTransport not safe for reuse after reading corrupted input

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4326?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16177489#comment-16177489 ] ASF GitHub Bot commented on THRIFT-4326: Github user benweint commented on the is

[GitHub] thrift pull request #1352: THRIFT-4326 Allow safer reuse of BufferedTranspor...

2017-09-22 Thread benweint
Github user benweint closed the pull request at: https://github.com/apache/thrift/pull/1352 ---

[jira] [Commented] (THRIFT-4326) Ruby BufferedTransport not safe for reuse after reading corrupted input

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4326?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16177490#comment-16177490 ] ASF GitHub Bot commented on THRIFT-4326: Github user benweint closed the pull req

[GitHub] thrift issue #1352: THRIFT-4326 Allow safer reuse of BufferedTransport insta...

2017-09-22 Thread benweint
Github user benweint commented on the issue: https://github.com/apache/thrift/pull/1352 Ok, I've finally figured out how to reproduce that test failure locally on macOS. For posterity, here's what I did: ``` PATH="/usr/local/opt/bison/bin:$PATH" ./configure --with-openssl

[jira] [Commented] (THRIFT-4329) c_glib Doesn't have a multiplexed processor

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16177363#comment-16177363 ] ASF GitHub Bot commented on THRIFT-4329: Github user jeking3 commented on the iss

[GitHub] thrift issue #1361: THRIFT-4329: Implement multiplexed processor that matche...

2017-09-22 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1361 Hmm, I ran cross tests locally and multi isn't being exercised for c_glib server: ``` root@ecb350ad7c01:/thrift/src# test/test.py --server c_glib --client java Apache Thrift - Integration

[jira] [Commented] (THRIFT-4329) c_glib Doesn't have a multiplexed processor

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16177362#comment-16177362 ] ASF GitHub Bot commented on THRIFT-4329: Github user jeking3 commented on the iss

[GitHub] thrift issue #1361: THRIFT-4329: Implement multiplexed processor that matche...

2017-09-22 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1361 This passed for me locally on a full cross, so I'll merge it. ---

[jira] [Commented] (THRIFT-4329) c_glib Doesn't have a multiplexed processor

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16177270#comment-16177270 ] ASF GitHub Bot commented on THRIFT-4329: Github user jeking3 commented on the iss

[GitHub] thrift issue #1361: THRIFT-4329: Implement multiplexed processor that matche...

2017-09-22 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1361 @gadLinux I am running the cross test suite locally. The errors in the build appear to be in c_glib... ---

[jira] [Updated] (THRIFT-4329) c_glib Doesn't have a multiplexed processor

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4329?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III updated THRIFT-4329: --- Fix Version/s: (was: 1.0) > c_glib Doesn't have a multiplexed processor > ---

[jira] [Updated] (THRIFT-4329) c_glib Doesn't have a multiplexed processor

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4329?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III updated THRIFT-4329: --- Affects Version/s: (was: 1.0) 0.10.0 > c_glib Doesn't have

[GitHub] thrift issue #1263: D lib updated for x64 build mode on windows

2017-09-22 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1263 This should be rebased on the current master as I have updated the docker files. Perhaps if you could open a Jira ticket for this issue and rebase on master and force push, we can get a more recent

[GitHub] thrift issue #1075: THRIFT-3916 Throw proper errors from JS, not strings

2017-09-22 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1075 @SimenB if you are interested in resurrecting this let me know - rebase on current master and push to CI will run all our tests. ---

[jira] [Commented] (THRIFT-3916) Errors thrown from JavaScript client is strings and not errors

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3916?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16177263#comment-16177263 ] ASF GitHub Bot commented on THRIFT-3916: Github user jeking3 commented on the iss

[jira] [Commented] (THRIFT-3830) Add detail exception message between server-side and client-side

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3830?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16177259#comment-16177259 ] ASF GitHub Bot commented on THRIFT-3830: Github user jeking3 commented on the iss

[GitHub] thrift issue #1012: THRIFT-3830

2017-09-22 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1012 @jfarrell this can be closed; it was resolved as won't fix. ---

[GitHub] thrift issue #1015: Update connection.js

2017-09-22 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1015 @jfarrell this PR can be closed; the line of code removed here was already done by another commit. ---

[jira] [Commented] (THRIFT-3821) TMemoryBuffer buffer may overflow when resizing

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3821?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16177257#comment-16177257 ] James E. King, III commented on THRIFT-3821: I checked in a small change to m

[jira] [Commented] (THRIFT-3821) TMemoryBuffer buffer may overflow when resizing

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3821?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16177255#comment-16177255 ] ASF GitHub Bot commented on THRIFT-3821: Github user asfgit closed the pull reque

[GitHub] thrift pull request #1369: THRIFT-3821: make memory buffer size configurable

2017-09-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/thrift/pull/1369 ---

[GitHub] thrift pull request #1370: THRIFT-3766: add zlib getUnderlyingTransport meth...

2017-09-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/thrift/pull/1370 ---

[jira] [Commented] (THRIFT-3766) Add getUnderlyingTransport() to TZlibTransport

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3766?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16177251#comment-16177251 ] ASF GitHub Bot commented on THRIFT-3766: Github user asfgit closed the pull reque

[jira] [Resolved] (THRIFT-3766) Add getUnderlyingTransport() to TZlibTransport

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3766?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III resolved THRIFT-3766. Resolution: Fixed Fix Version/s: 0.11.0 > Add getUnderlyingTransport() to TZ

[jira] [Commented] (THRIFT-4329) c_glib Doesn't have a multiplexed processor

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16177210#comment-16177210 ] ASF GitHub Bot commented on THRIFT-4329: Github user gadLinux commented on the is

[GitHub] thrift issue #1361: THRIFT-4329: Implement multiplexed processor that matche...

2017-09-22 Thread gadLinux
Github user gadLinux commented on the issue: https://github.com/apache/thrift/pull/1361 Perfect, now fails on things not related to my changes. ---

[jira] [Commented] (THRIFT-3766) Add getUnderlyingTransport() to TZlibTransport

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3766?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16177160#comment-16177160 ] ASF GitHub Bot commented on THRIFT-3766: Github user jeking3 commented on the iss

[GitHub] thrift issue #1370: THRIFT-3766: add zlib getUnderlyingTransport method

2017-09-22 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1370 I've been really busy. :) ---

Re: Proposal: Retire the "Patch" submission method

2017-09-22 Thread Jens Geyer
Hi, haven 't read your changes, but although I made the same proposal a while ago here are my thoughts: a) I'm against removing the patch method entirely b) would nevertheless make clear that we absolutely prefer pull requests c) make PRs mandatory only if the patch is beyond a certain size, i.e

[jira] [Commented] (THRIFT-3766) Add getUnderlyingTransport() to TZlibTransport

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3766?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16177123#comment-16177123 ] ASF GitHub Bot commented on THRIFT-3766: Github user Jens-G commented on the issu

[GitHub] thrift issue #1370: THRIFT-3766: add zlib getUnderlyingTransport method

2017-09-22 Thread Jens-G
Github user Jens-G commented on the issue: https://github.com/apache/thrift/pull/1370 Yeah, test. I got only 2 mails this time. Your comment generated 3 mails. I had cases where I got up to 7 mails for one github comment. ---

[jira] [Commented] (THRIFT-3766) Add getUnderlyingTransport() to TZlibTransport

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3766?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16177118#comment-16177118 ] ASF GitHub Bot commented on THRIFT-3766: Github user jeking3 commented on the iss

[GitHub] thrift issue #1370: THRIFT-3766: add zlib getUnderlyingTransport method

2017-09-22 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1370 @Jens-G test? ---

[jira] [Commented] (THRIFT-3766) Add getUnderlyingTransport() to TZlibTransport

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3766?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16177112#comment-16177112 ] ASF GitHub Bot commented on THRIFT-3766: Github user Jens-G commented on the issu

[GitHub] thrift issue #1370: THRIFT-3766: add zlib getUnderlyingTransport method

2017-09-22 Thread Jens-G
Github user Jens-G commented on the issue: https://github.com/apache/thrift/pull/1370 Test ---

[jira] [Commented] (THRIFT-4329) c_glib Doesn't have a multiplexed processor

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16177110#comment-16177110 ] ASF GitHub Bot commented on THRIFT-4329: Github user gadLinux commented on the is

[GitHub] thrift issue #1361: THRIFT-4329: Implement multiplexed processor that matche...

2017-09-22 Thread gadLinux
Github user gadLinux commented on the issue: https://github.com/apache/thrift/pull/1361 It seems the protocol decorator was removed, I suppose I copied and pasted wrong... I always forget that the build system is Cmake and forget to test it against it. ---

[jira] [Assigned] (THRIFT-2740) Perl coding standards

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2740?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III reassigned THRIFT-2740: -- Assignee: James E. King, III > Perl coding standards > - >

[jira] [Commented] (THRIFT-2740) Perl coding standards

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2740?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176960#comment-16176960 ] James E. King, III commented on THRIFT-2740: [~hcorg] Sufficient to resolve t

[jira] [Updated] (THRIFT-4340) Erlang fix a crash on client close

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4340?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III updated THRIFT-4340: --- Description: When a client closes a connection to a framed server the server was cras

[jira] [Assigned] (THRIFT-4340) Erlang fix a crash on client close

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4340?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III reassigned THRIFT-4340: -- Assignee: Anthony Molinaro (was: James E. King, III) > Erlang fix a crash on

[jira] [Commented] (THRIFT-4339) Thrift Framed Transport in Erlang crashes server when client disconnects

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4339?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176936#comment-16176936 ] James E. King, III commented on THRIFT-4339: In the future add the Jira item

[GitHub] thrift pull request #1370: THRIFT-3766: add zlib getUnderlyingTransport meth...

2017-09-22 Thread jeking3
GitHub user jeking3 opened a pull request: https://github.com/apache/thrift/pull/1370 THRIFT-3766: add zlib getUnderlyingTransport method Client: C++ This closes #1370 You can merge this pull request into a Git repository by running: $ git pull https://github.com/jekin

[jira] [Commented] (THRIFT-3766) Add getUnderlyingTransport() to TZlibTransport

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3766?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176917#comment-16176917 ] ASF GitHub Bot commented on THRIFT-3766: GitHub user jeking3 opened a pull reques

[jira] [Assigned] (THRIFT-3766) Add getUnderlyingTransport() to TZlibTransport

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3766?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III reassigned THRIFT-3766: -- Assignee: James E. King, III > Add getUnderlyingTransport() to TZlibTransport

[jira] [Closed] (THRIFT-4258) Boost/std thread wrapping faultiness

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4258?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III closed THRIFT-4258. -- > Boost/std thread wrapping faultiness > > >

[jira] [Resolved] (THRIFT-4258) Boost/std thread wrapping faultiness

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4258?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III resolved THRIFT-4258. Resolution: Won't Fix In the current product, selection of the thread mechanism is

[jira] [Assigned] (THRIFT-4258) Boost/std thread wrapping faultiness

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4258?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III reassigned THRIFT-4258: -- Assignee: James E. King, III > Boost/std thread wrapping faultiness >

[jira] [Closed] (THRIFT-4339) Thrift Framed Transport in Erlang crashes server when client disconnects

2017-09-22 Thread Anthony Molinaro (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4339?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Anthony Molinaro closed THRIFT-4339. Resolution: Duplicate > Thrift Framed Transport in Erlang crashes server when client discon

[jira] [Commented] (THRIFT-4339) Thrift Framed Transport in Erlang crashes server when client disconnects

2017-09-22 Thread Anthony Molinaro (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4339?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176886#comment-16176886 ] Anthony Molinaro commented on THRIFT-4339: -- Patch is attached here and there was

[jira] [Commented] (THRIFT-3238) Fix TNamedPipeServer can be interrupted by faulty client

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176887#comment-16176887 ] James E. King, III commented on THRIFT-3238: THRIFT-4283 might be relevant he

[jira] [Commented] (THRIFT-3481) introduce TSimpleMemoryTransport which wraps a customer owned memory chunk to conduct both read and write operations

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176882#comment-16176882 ] James E. King, III commented on THRIFT-3481: I agree with [~nsuke] here - mod

[jira] [Updated] (THRIFT-4339) Thrift Framed Transport in Erlang crashes server when client disconnects

2017-09-22 Thread Anthony Molinaro (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4339?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Anthony Molinaro updated THRIFT-4339: - Attachment: erl_framed_transport_fix.patch > Thrift Framed Transport in Erlang crashes se

[jira] [Commented] (THRIFT-4339) Thrift Framed Transport in Erlang crashes server when client disconnects

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4339?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176879#comment-16176879 ] ASF GitHub Bot commented on THRIFT-4339: Github user djnym commented on the issue

[GitHub] thrift issue #1367: Fix a crash on client close

2017-09-22 Thread djnym
Github user djnym commented on the issue: https://github.com/apache/thrift/pull/1367 I added a jira ticket yesterday https://issues.apache.org/jira/browse/THRIFT-4339 ---

[jira] [Commented] (THRIFT-3821) TMemoryBuffer buffer may overflow when resizing

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3821?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176872#comment-16176872 ] ASF GitHub Bot commented on THRIFT-3821: GitHub user jeking3 opened a pull reques

[GitHub] thrift pull request #1369: THRIFT-3821: make memory buffer size configurable

2017-09-22 Thread jeking3
GitHub user jeking3 opened a pull request: https://github.com/apache/thrift/pull/1369 THRIFT-3821: make memory buffer size configurable so unit test doesn't need 2GB to run; also added a unit test to prove THRIFT-3480 alternative You can merge this pull request into a Git repo

[jira] [Closed] (THRIFT-3480) offer a simple approach to compute byte size of the serialized object

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III closed THRIFT-3480. -- > offer a simple approach to compute byte size of the serialized object > -

[jira] [Resolved] (THRIFT-3480) offer a simple approach to compute byte size of the serialized object

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III resolved THRIFT-3480. Resolution: Won't Fix This can be achieved with a TMemoryBuffer (I wrote a unit tes

[jira] [Assigned] (THRIFT-3480) offer a simple approach to compute byte size of the serialized object

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-3480?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III reassigned THRIFT-3480: -- Assignee: James E. King, III > offer a simple approach to compute byte size of

[jira] [Commented] (THRIFT-1761) Make generated exceptions have a common unique parent

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-1761?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176737#comment-16176737 ] James E. King, III commented on THRIFT-1761: Any interest in reviving this, o

[jira] [Commented] (THRIFT-4283) TNamedPipeServer race condition in interrupt

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4283?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176735#comment-16176735 ] ASF GitHub Bot commented on THRIFT-4283: Github user asfgit closed the pull reque

[jira] [Resolved] (THRIFT-4283) TNamedPipeServer race condition in interrupt

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4283?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III resolved THRIFT-4283. Resolution: Fixed Committed, thanks. > TNamedPipeServer race condition in interrup

[GitHub] thrift pull request #1368: THRIFT-4283: fix TNamedPipeServer race condition ...

2017-09-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/thrift/pull/1368 ---

[jira] [Assigned] (THRIFT-4283) TNamedPipeServer race condition in interrupt

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4283?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III reassigned THRIFT-4283: -- Assignee: James E. King, III > TNamedPipeServer race condition in interrupt >

[jira] [Commented] (THRIFT-4283) TNamedPipeServer race condition in interrupt

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4283?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176665#comment-16176665 ] ASF GitHub Bot commented on THRIFT-4283: GitHub user jeking3 opened a pull reques

[GitHub] thrift pull request #1368: THRIFT-4283: fix TNamedPipeServer race condition ...

2017-09-22 Thread jeking3
GitHub user jeking3 opened a pull request: https://github.com/apache/thrift/pull/1368 THRIFT-4283: fix TNamedPipeServer race condition in interrupt Client: C++ Patch: jn.g...@evs.com You can merge this pull request into a Git repository by running: $ git pull https://github

[jira] [Commented] (THRIFT-4258) Boost/std thread wrapping faultiness

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4258?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176637#comment-16176637 ] James E. King, III commented on THRIFT-4258: The current implementation suppo

[jira] [Created] (THRIFT-4342) Support ruby rspec 3

2017-09-22 Thread James E. King, III (JIRA)
James E. King, III created THRIFT-4342: -- Summary: Support ruby rspec 3 Key: THRIFT-4342 URL: https://issues.apache.org/jira/browse/THRIFT-4342 Project: Thrift Issue Type: Improvement

[jira] [Commented] (THRIFT-4289) Thrift RSpec test suite fails with Ruby 2.4.x due to Fixnum deprecation

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4289?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176603#comment-16176603 ] James E. King, III commented on THRIFT-4289: Would you be able to submit a gi

[jira] [Commented] (THRIFT-4329) c_glib Doesn't have a multiplexed processor

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176565#comment-16176565 ] ASF GitHub Bot commented on THRIFT-4329: Github user jeking3 commented on the iss

[jira] [Updated] (THRIFT-2913) Ruby Server Thrift::ThreadPoolServer should serve inside a thread

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2913?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III updated THRIFT-2913: --- Affects Version/s: 0.10.0 > Ruby Server Thrift::ThreadPoolServer should serve inside

[GitHub] thrift issue #1361: THRIFT-4329: Implement multiplexed processor that matche...

2017-09-22 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1361 Ruby error in build job 4 is https://issues.apache.org/jira/browse/THRIFT-2913. c_glib unit tests ran so we can overlook this failure. ---

[jira] [Updated] (THRIFT-2913) Ruby Server Thrift::ThreadPoolServer should serve inside a thread

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2913?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III updated THRIFT-2913: --- Environment: openSUSE 13.2, Ruby 2.1.0 also in docker ubuntu-xenial image, per CI bui

[jira] [Commented] (THRIFT-2913) Ruby Server Thrift::ThreadPoolServer should serve inside a thread

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-2913?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176561#comment-16176561 ] James E. King, III commented on THRIFT-2913: Appeared again in a pull request

[jira] [Commented] (THRIFT-4329) c_glib Doesn't have a multiplexed processor

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176551#comment-16176551 ] ASF GitHub Bot commented on THRIFT-4329: Github user jeking3 commented on the iss

[GitHub] thrift issue #1361: THRIFT-4329: Implement multiplexed processor that matche...

2017-09-22 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1361 Build job 4 failed in ruby testing. Looking for a match in PRs or Jira. ---

[jira] [Commented] (THRIFT-4264) PHP - Support both shared & static linking of sockets library

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4264?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176543#comment-16176543 ] ASF GitHub Bot commented on THRIFT-4264: Github user asfgit closed the pull reque

[GitHub] thrift pull request #1315: THRIFT-4264: Fix PHP tests requiring sockets.so

2017-09-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/thrift/pull/1315 ---

[jira] [Resolved] (THRIFT-4264) PHP - Support both shared & static linking of sockets library

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4264?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III resolved THRIFT-4264. Resolution: Fixed Fix Version/s: 0.11.0 Committed, thanks. > PHP - Support

[jira] [Assigned] (THRIFT-4264) PHP - Support both shared & static linking of sockets library

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4264?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] James E. King, III reassigned THRIFT-4264: -- Assignee: James E. King, III > PHP - Support both shared & static linking of s

[jira] [Updated] (THRIFT-4341) No WARNING or ERROR when included .thrift file is not found while running thrift-maven-plugin

2017-09-22 Thread Merter Sualp (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4341?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Merter Sualp updated THRIFT-4341: - Priority: Minor (was: Major) Summary: No WARNING or ERROR when included .thrift file is not

[jira] [Updated] (THRIFT-4341) No WARNING or ERROR when included .thrift file is not found

2017-09-22 Thread Merter Sualp (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4341?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Merter Sualp updated THRIFT-4341: - Description: While using thrift-maven-plugin, we define our Javascript execution as below: {code

Proposal: Retire the "Patch" submission method

2017-09-22 Thread James E. King, III
We've been using github for a while now. At this point everyone should be expected to submit changes to the project through github. Tickets in Jira that have patches without a pull request experience significant delays. I would prefer to see the Jira option for "Patch Available" removed. Folks c

[jira] [Updated] (THRIFT-4341) No WARNING or ERROR when included .thrift file is not found

2017-09-22 Thread Merter Sualp (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4341?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Merter Sualp updated THRIFT-4341: - Attachment: no_thrift_js_reported.jpg > No WARNING or ERROR when included .thrift file is not fou

[jira] [Updated] (THRIFT-4341) No WARNING or ERROR when included .thrift file is not found

2017-09-22 Thread Merter Sualp (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4341?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Merter Sualp updated THRIFT-4341: - Description: While using thrift-maven-plugin, we define our Javascript execution as below: {code

[jira] [Created] (THRIFT-4341) No WARNING or ERROR when included .thrift file is not found

2017-09-22 Thread Merter Sualp (JIRA)
Merter Sualp created THRIFT-4341: Summary: No WARNING or ERROR when included .thrift file is not found Key: THRIFT-4341 URL: https://issues.apache.org/jira/browse/THRIFT-4341 Project: Thrift

[jira] [Updated] (THRIFT-4341) No WARNING or ERROR when included .thrift file is not found

2017-09-22 Thread Merter Sualp (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4341?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Merter Sualp updated THRIFT-4341: - Description: While using thrift-maven-plugin, we define our Javascript execution as below: {code

[jira] [Commented] (THRIFT-4283) TNamedPipeServer race condition in interrupt

2017-09-22 Thread JIRA
[ https://issues.apache.org/jira/browse/THRIFT-4283?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176430#comment-16176430 ] Jean-Noël Goor commented on THRIFT-4283: I would prefer you to do it if you don't

[GitHub] thrift issue #1361: THRIFT-4329: Implement multiplexed processor that matche...

2017-09-22 Thread gadLinux
Github user gadLinux commented on the issue: https://github.com/apache/thrift/pull/1361 I just went back in history and pushed again after rebase. Now it should be ok ---

[jira] [Commented] (THRIFT-4329) c_glib Doesn't have a multiplexed processor

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176409#comment-16176409 ] ASF GitHub Bot commented on THRIFT-4329: Github user gadLinux commented on the is

[jira] [Commented] (THRIFT-4264) PHP - Support both shared & static linking of sockets library

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4264?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176404#comment-16176404 ] ASF GitHub Bot commented on THRIFT-4264: Github user norrs commented on the issue

[GitHub] thrift issue #1315: THRIFT-4264: Fix PHP tests requiring sockets.so

2017-09-22 Thread norrs
Github user norrs commented on the issue: https://github.com/apache/thrift/pull/1315 @jeking3 : Done; let's see how it goes this time. ---

[jira] [Commented] (THRIFT-4283) TNamedPipeServer race condition in interrupt

2017-09-22 Thread James E. King, III (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4283?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176389#comment-16176389 ] James E. King, III commented on THRIFT-4283: Hi, thanks for the patch. Someo

[jira] [Updated] (THRIFT-4283) TNamedPipeServer race condition in interrupt

2017-09-22 Thread JIRA
[ https://issues.apache.org/jira/browse/THRIFT-4283?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jean-Noël Goor updated THRIFT-4283: --- Fix Version/s: 0.11.0 > TNamedPipeServer race condition in interrupt > --

[jira] [Commented] (THRIFT-4329) c_glib Doesn't have a multiplexed processor

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176358#comment-16176358 ] ASF GitHub Bot commented on THRIFT-4329: Github user jeking3 commented on the iss

[GitHub] thrift issue #1361: THRIFT-4329: Implement multiplexed processor that matche...

2017-09-22 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1361 Try this: ``` git checkout master git pull upstream master git checkout -b THRIFT-4329-take-2 git cherry-pick 1d1d9d8 git cherry-pick 0aedc8a git cherry-pick 3df26ec

[jira] [Commented] (THRIFT-4329) c_glib Doesn't have a multiplexed processor

2017-09-22 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/THRIFT-4329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16176356#comment-16176356 ] ASF GitHub Bot commented on THRIFT-4329: Github user jeking3 commented on the iss

[GitHub] thrift issue #1361: THRIFT-4329: Implement multiplexed processor that matche...

2017-09-22 Thread jeking3
Github user jeking3 commented on the issue: https://github.com/apache/thrift/pull/1361 Doesn't look like a rebase, unfortunately. What I would suggest is that you refresh your master from upstream, branch off it, cherry-pick the original commits you made in order, then squash them

[GitHub] thrift pull request #1367: Fix a crash on client close

2017-09-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/thrift/pull/1367 ---

  1   2   >