[jira] [Commented] (THRIFT-3768) TThreadedServer may crash if it is destroyed immediately after it returns from serve()

[ https://issues.apache.org/jira/browse/THRIFT-3768?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15226689#comment-15226689 ] Ben Craig commented on THRIFT-3768: --- ThreadManager is a mess. When the C++11 revolutio

[jira] [Commented] (THRIFT-3768) TThreadedServer may crash if it is destroyed immediately after it returns from serve()

[ https://issues.apache.org/jira/browse/THRIFT-3768?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15220756#comment-15220756 ] Ben Craig commented on THRIFT-3768: --- You probably will need two vectors. You can servi

[jira] [Commented] (THRIFT-3768) TThreadedServer may crash if it is destroyed immediately after it returns from serve()

[ https://issues.apache.org/jira/browse/THRIFT-3768?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15220595#comment-15220595 ] Ben Craig commented on THRIFT-3768: --- Not sure how this will get reflected in code, but

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

[ https://issues.apache.org/jira/browse/THRIFT-3238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15213225#comment-15213225 ] Ben Craig commented on THRIFT-3238: --- ugh, makes sense. The remainder of the attached p

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

[ https://issues.apache.org/jira/browse/THRIFT-3238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15211083#comment-15211083 ] Ben Craig commented on THRIFT-3238: --- The provided patches touch a lot more code than ne

[jira] [Commented] (THRIFT-3371) Abstract namespace Unix domain sockets broken in C++

[ https://issues.apache.org/jira/browse/THRIFT-3371?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14951302#comment-14951302 ] Ben Craig commented on THRIFT-3371: --- +1 > Abstract namespace Unix domain sockets broke

[jira] [Commented] (THRIFT-3237) Fix TNamedPipeServer::createNamedPipe memory leak

[ https://issues.apache.org/jira/browse/THRIFT-3237?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14944140#comment-14944140 ] Ben Craig commented on THRIFT-3237: --- +1 > Fix TNamedPipeServer::createNamedPipe memory

[jira] [Commented] (THRIFT-3228) Fix TAutoOverlapThread may reference released memory

[ https://issues.apache.org/jira/browse/THRIFT-3228?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14934471#comment-14934471 ] Ben Craig commented on THRIFT-3228: --- There are basically two major options... 1. Break

[jira] [Resolved] (THRIFT-3270) reusing TNonblockingServer::TConnection cause dirty TSocket

[ https://issues.apache.org/jira/browse/THRIFT-3270?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig resolved THRIFT-3270. --- Resolution: Fixed Thanks for the patch Jesse! > reusing TNonblockingServer::TConnection cause dirty

[jira] [Resolved] (THRIFT-3252) Missing TConcurrentClientSyncInfo.h in cpp Makefile, so doesn't install

[ https://issues.apache.org/jira/browse/THRIFT-3252?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig resolved THRIFT-3252. --- Resolution: Fixed Assignee: Adam Beberg Fix Version/s: 0.9.3 > Missing TConcurrentCli

[jira] [Commented] (THRIFT-3252) Missing TConcurrentClientSyncInfo.h in cpp Makefile, so doesn't install

[ https://issues.apache.org/jira/browse/THRIFT-3252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14637987#comment-14637987 ] Ben Craig commented on THRIFT-3252: --- Committed. Thanks! > Missing TConcurrentClientSy

[jira] [Resolved] (THRIFT-3247) Generate a C++ thread-safe client

[ https://issues.apache.org/jira/browse/THRIFT-3247?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig resolved THRIFT-3247. --- Resolution: Fixed Fix Version/s: 0.9.3 > Generate a C++ thread-safe client > -

[jira] [Resolved] (THRIFT-3224) Fix TNamedPipeServer unpredictable behavior on accept

[ https://issues.apache.org/jira/browse/THRIFT-3224?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig resolved THRIFT-3224. --- Resolution: Fixed > Fix TNamedPipeServer unpredictable behavior on accept > -

[jira] [Resolved] (THRIFT-3225) Fix TPipeServer unpredictable behavior on interrupt()

[ https://issues.apache.org/jira/browse/THRIFT-3225?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig resolved THRIFT-3225. --- Resolution: Fixed > Fix TPipeServer unpredictable behavior on interrupt() > -

[jira] [Commented] (THRIFT-3225) Fix TPipeServer unpredictable behavior on interrupt()

[ https://issues.apache.org/jira/browse/THRIFT-3225?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14629697#comment-14629697 ] Ben Craig commented on THRIFT-3225: --- Resolved in the same patch as THRIFT-3224 > Fix T

[jira] [Created] (THRIFT-3247) Generate a C++ thread-safe client

Ben Craig created THRIFT-3247: - Summary: Generate a C++ thread-safe client Key: THRIFT-3247 URL: https://issues.apache.org/jira/browse/THRIFT-3247 Project: Thrift Issue Type: Improvement

[jira] [Closed] (THRIFT-3231) CPP: Limit recursion depth to 64

[ https://issues.apache.org/jira/browse/THRIFT-3231?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig closed THRIFT-3231. - > CPP: Limit recursion depth to 64 > > > Key: THRIFT-3231 >

[jira] [Commented] (THRIFT-3246) Reduce the number of trivial warnings in Windows C++ CMake builds

[ https://issues.apache.org/jira/browse/THRIFT-3246?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14628511#comment-14628511 ] Ben Craig commented on THRIFT-3246: --- This was mostly on the test side, and I don't reca

[jira] [Resolved] (THRIFT-3246) Reduce the number of trivial warnings in Windows C++ CMake builds

[ https://issues.apache.org/jira/browse/THRIFT-3246?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig resolved THRIFT-3246. --- Resolution: Fixed > Reduce the number of trivial warnings in Windows C++ CMake builds > -

[jira] [Resolved] (THRIFT-2858) Enable header field case insensitive match in THttpServer

[ https://issues.apache.org/jira/browse/THRIFT-2858?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig resolved THRIFT-2858. --- Resolution: Fixed > Enable header field case insensitive match in THttpServer > -

[jira] [Reopened] (THRIFT-2858) Enable header field case insensitive match in THttpServer

[ https://issues.apache.org/jira/browse/THRIFT-2858?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig reopened THRIFT-2858: --- This breaks the windows build. Windows doesn't have a strncasecmp or strcasestr. Pull request will be

[jira] [Created] (THRIFT-3246) Reduce the number of trivial warnings in Windows C++ CMake builds

Ben Craig created THRIFT-3246: - Summary: Reduce the number of trivial warnings in Windows C++ CMake builds Key: THRIFT-3246 URL: https://issues.apache.org/jira/browse/THRIFT-3246 Project: Thrift

[jira] [Closed] (THRIFT-3245) Tutorials can't be disabled when using CMake

[ https://issues.apache.org/jira/browse/THRIFT-3245?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig closed THRIFT-3245. - > Tutorials can't be disabled when using CMake > > >

[jira] [Resolved] (THRIFT-3245) Tutorials can't be disabled when using CMake

[ https://issues.apache.org/jira/browse/THRIFT-3245?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig resolved THRIFT-3245. --- Resolution: Fixed Assignee: Pascal Bach Fix Version/s: 0.9.2 > Tutorials can't be dis

[jira] [Commented] (THRIFT-3245) Tutorials can't be disabled when using CMake

[ https://issues.apache.org/jira/browse/THRIFT-3245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14628038#comment-14628038 ] Ben Craig commented on THRIFT-3245: --- Thanks for the pull request Pascal. Once Travis C

[jira] [Commented] (THRIFT-3224) Fix TNamedPipeServer unpredictable behavior on accept

[ https://issues.apache.org/jira/browse/THRIFT-3224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14627073#comment-14627073 ] Ben Craig commented on THRIFT-3224: --- [~pjanicki] I have updated my pull request. It ad

[jira] [Commented] (THRIFT-3206) Fix Visual Studio build failure due 'pthread_self': identifier not found

[ https://issues.apache.org/jira/browse/THRIFT-3206?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14626544#comment-14626544 ] Ben Craig commented on THRIFT-3206: --- This is only an issue with older versions of open

[jira] [Commented] (THRIFT-3225) Fix TPipeServer unpredictable behavior on interrupt()

[ https://issues.apache.org/jira/browse/THRIFT-3225?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14621624#comment-14621624 ] Ben Craig commented on THRIFT-3225: --- This might be right. I'm going to try to dig up (

[jira] [Commented] (THRIFT-3221) Create a tool to audit network compatibility between two .thrift files

[ https://issues.apache.org/jira/browse/THRIFT-3221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14621618#comment-14621618 ] Ben Craig commented on THRIFT-3221: --- should be fixed now. > Create a tool to audit net

[jira] [Commented] (THRIFT-3224) Fix TNamedPipeServer unpredictable behavior on accept

[ https://issues.apache.org/jira/browse/THRIFT-3224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14621612#comment-14621612 ] Ben Craig commented on THRIFT-3224: --- updated my pull request. Take a look at that when

[jira] [Commented] (THRIFT-3221) Create a tool to audit network compatibility between two .thrift files

[ https://issues.apache.org/jira/browse/THRIFT-3221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14621100#comment-14621100 ] Ben Craig commented on THRIFT-3221: --- Did this break the cmake build, the autotools buil

[jira] [Commented] (THRIFT-3224) Fix TNamedPipeServer unpredictable behavior on accept

[ https://issues.apache.org/jira/browse/THRIFT-3224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14620524#comment-14620524 ] Ben Craig commented on THRIFT-3224: --- I agree that TAutoHandle references can make this

[jira] [Resolved] (THRIFT-3231) CPP: Limit recursion depth to 64

[ https://issues.apache.org/jira/browse/THRIFT-3231?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig resolved THRIFT-3231. --- Resolution: Fixed > CPP: Limit recursion depth to 64 > > >

[jira] [Commented] (THRIFT-3228) Fix TAutoOverlapThread may reference released memory

[ https://issues.apache.org/jira/browse/THRIFT-3228?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14620491#comment-14620491 ] Ben Craig commented on THRIFT-3228: --- A lot of my uses of Thrift involve multiple users

[jira] [Closed] (THRIFT-3217) Provide a little endian variant of the binary protocol in C++

[ https://issues.apache.org/jira/browse/THRIFT-3217?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig closed THRIFT-3217. - > Provide a little endian variant of the binary protocol in C++ > ---

[jira] [Resolved] (THRIFT-3221) Create a tool to audit network compatibility between two .thrift files

[ https://issues.apache.org/jira/browse/THRIFT-3221?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig resolved THRIFT-3221. --- Resolution: Fixed > Create a tool to audit network compatibility between two .thrift files >

[jira] [Closed] (THRIFT-3221) Create a tool to audit network compatibility between two .thrift files

[ https://issues.apache.org/jira/browse/THRIFT-3221?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig closed THRIFT-3221. - > Create a tool to audit network compatibility between two .thrift files > --

[jira] [Resolved] (THRIFT-3217) Provide a little endian variant of the binary protocol in C++

[ https://issues.apache.org/jira/browse/THRIFT-3217?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig resolved THRIFT-3217. --- Resolution: Fixed Fix Version/s: 0.9.3 > Provide a little endian variant of the binary protoco

[jira] [Created] (THRIFT-3231) CPP: Limit recursion depth to 64

Ben Craig created THRIFT-3231: - Summary: CPP: Limit recursion depth to 64 Key: THRIFT-3231 URL: https://issues.apache.org/jira/browse/THRIFT-3231 Project: Thrift Issue Type: Bug Compone

[jira] [Commented] (THRIFT-3228) Fix TAutoOverlapThread may reference released memory

[ https://issues.apache.org/jira/browse/THRIFT-3228?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14618801#comment-14618801 ] Ben Craig commented on THRIFT-3228: --- Function level statics are not thread safe in Visu

[jira] [Commented] (THRIFT-3228) Fix TAutoOverlapThread may reference released memory

[ https://issues.apache.org/jira/browse/THRIFT-3228?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14618636#comment-14618636 ] Ben Craig commented on THRIFT-3228: --- This is a documented limitation. From TPipe.h: /*

[jira] [Commented] (THRIFT-3226) Fix TNamedPipeServer trapped in loop on accept

[ https://issues.apache.org/jira/browse/THRIFT-3226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14617351#comment-14617351 ] Ben Craig commented on THRIFT-3226: --- Don't try to use closed transports. Calling inter

[jira] [Commented] (THRIFT-3224) Fix TNamedPipeServer unpredictable behavior on accept

[ https://issues.apache.org/jira/browse/THRIFT-3224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14617344#comment-14617344 ] Ben Craig commented on THRIFT-3224: --- This patch leaks pipes if the TPipe ctor throws.

[jira] [Commented] (THRIFT-3225) Fix TPipeServer unpredictable behavior on close

[ https://issues.apache.org/jira/browse/THRIFT-3225?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14617329#comment-14617329 ] Ben Craig commented on THRIFT-3225: --- doh! please back this out. I'm reviewing some of

[jira] [Commented] (THRIFT-3225) Fix TPipeServer unpredictable behavior on close

[ https://issues.apache.org/jira/browse/THRIFT-3225?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14617321#comment-14617321 ] Ben Craig commented on THRIFT-3225: --- -1 close() doesn't need to be / shouldn't be thre

[jira] [Comment Edited] (THRIFT-3221) Create a tool to audit network compatibility between two .thrift files

[ https://issues.apache.org/jira/browse/THRIFT-3221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14614374#comment-14614374 ] Ben Craig edited comment on THRIFT-3221 at 7/5/15 8:19 PM: --- The

[jira] [Commented] (THRIFT-3221) Create a tool to audit network compatibility between two .thrift files

[ https://issues.apache.org/jira/browse/THRIFT-3221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14614374#comment-14614374 ] Ben Craig commented on THRIFT-3221: --- The compiler help says how to use it, but right no

[jira] [Closed] (THRIFT-3219) Provide a C++ tutorial on server-side IP logging and per-connection state

[ https://issues.apache.org/jira/browse/THRIFT-3219?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig closed THRIFT-3219. - > Provide a C++ tutorial on server-side IP logging and per-connection state > ---

[jira] [Resolved] (THRIFT-3219) Provide a C++ tutorial on server-side IP logging and per-connection state

[ https://issues.apache.org/jira/browse/THRIFT-3219?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig resolved THRIFT-3219. --- Resolution: Fixed Assignee: Ben Craig > Provide a C++ tutorial on server-side IP logging and pe

[jira] [Created] (THRIFT-3221) Create a tool to audit network compatibility between two .thrift files

Ben Craig created THRIFT-3221: - Summary: Create a tool to audit network compatibility between two .thrift files Key: THRIFT-3221 URL: https://issues.apache.org/jira/browse/THRIFT-3221 Project: Thrift

[jira] [Commented] (THRIFT-3217) Provide a little endian variant of the binary protocol in C++

[ https://issues.apache.org/jira/browse/THRIFT-3217?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14613505#comment-14613505 ] Ben Craig commented on THRIFT-3217: --- Beefed up the performance test a bit. Here's my l

[jira] [Commented] (THRIFT-3217) Provide a little endian variant of the binary protocol in C++

[ https://issues.apache.org/jira/browse/THRIFT-3217?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14613180#comment-14613180 ] Ben Craig commented on THRIFT-3217: --- I'll see if I can rustle up some ARM numbers. I'l

[jira] [Assigned] (THRIFT-3217) Provide a little endian variant of the binary protocol in C++

[ https://issues.apache.org/jira/browse/THRIFT-3217?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig reassigned THRIFT-3217: - Assignee: Ben Craig > Provide a little endian variant of the binary protocol in C++ > --

[jira] [Updated] (THRIFT-3217) Provide a little endian variant of the binary protocol in C++

[ https://issues.apache.org/jira/browse/THRIFT-3217?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig updated THRIFT-3217: -- Component/s: C++ - Library > Provide a little endian variant of the binary protocol in C++ > --

[jira] [Created] (THRIFT-3219) Provide a C++ tutorial on server-side IP logging and per-connection state

Ben Craig created THRIFT-3219: - Summary: Provide a C++ tutorial on server-side IP logging and per-connection state Key: THRIFT-3219 URL: https://issues.apache.org/jira/browse/THRIFT-3219 Project: Thrift

[jira] [Created] (THRIFT-3217) Provide a little endian variant of the binary protocol in C++

Ben Craig created THRIFT-3217: - Summary: Provide a little endian variant of the binary protocol in C++ Key: THRIFT-3217 URL: https://issues.apache.org/jira/browse/THRIFT-3217 Project: Thrift Iss

[jira] [Commented] (THRIFT-3038) Use of volatile in cpp library

[ https://issues.apache.org/jira/browse/THRIFT-3038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14562856#comment-14562856 ] Ben Craig commented on THRIFT-3038: --- You could take that path that the threading and fu

[jira] [Commented] (THRIFT-3038) Use of volatile in cpp library

[ https://issues.apache.org/jira/browse/THRIFT-3038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14561769#comment-14561769 ] Ben Craig commented on THRIFT-3038: --- The Thrift Cpp library does not currently require

[jira] [Commented] (THRIFT-1944) Binding to zero port

[ https://issues.apache.org/jira/browse/THRIFT-1944?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14532578#comment-14532578 ] Ben Craig commented on THRIFT-1944: --- To recap from earlier in the thread... " 1) soc

[jira] [Closed] (THRIFT-2150) Should windows/config.h define WIN32_LEAN_AND_MEAN?

[ https://issues.apache.org/jira/browse/THRIFT-2150?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig closed THRIFT-2150. - Resolution: Won't Fix This is a Microsoft build infrastructure issue that is external to the project wit

[jira] [Updated] (THRIFT-2441) Cannot shutdown TThreadedServer when clients are still connected

[ https://issues.apache.org/jira/browse/THRIFT-2441?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig updated THRIFT-2441: -- Assignee: James E. King, III (was: Ben Craig) > Cannot shutdown TThreadedServer when clients are still

[jira] [Resolved] (THRIFT-2441) Cannot shutdown TThreadedServer when clients are still connected

[ https://issues.apache.org/jira/browse/THRIFT-2441?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig resolved THRIFT-2441. --- Resolution: Fixed > Cannot shutdown TThreadedServer when clients are still connected > --

[jira] [Commented] (THRIFT-2441) Cannot shutdown TThreadedServer when clients are still connected

[ https://issues.apache.org/jira/browse/THRIFT-2441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14511098#comment-14511098 ] Ben Craig commented on THRIFT-2441: --- committed. Thanks jeking3! > Cannot shutdown TTh

[jira] [Closed] (THRIFT-2119) Compiling the Thrift C++ Library with Visual Studio 2005

[ https://issues.apache.org/jira/browse/THRIFT-2119?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ben Craig closed THRIFT-2119. - Resolution: Won't Fix See other comments. VS2005 is old enough that we don't feel the need to prioritiz

[jira] [Commented] (THRIFT-2119) Compiling the Thrift C++ Library with Visual Studio 2005

[ https://issues.apache.org/jira/browse/THRIFT-2119?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14503602#comment-14503602 ] Ben Craig commented on THRIFT-2119: --- I am fine dropping this as well. I will close. N

[jira] [Commented] (THRIFT-3081) C++ Consolidate client processing loops in TServers

[ https://issues.apache.org/jira/browse/THRIFT-3081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14490598#comment-14490598 ] Ben Craig commented on THRIFT-3081: --- I approve of the general direction of the recently

[jira] [Commented] (THRIFT-2441) Cannot shutdown TThreadedServer when clients are still connected

[ https://issues.apache.org/jira/browse/THRIFT-2441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14490579#comment-14490579 ] Ben Craig commented on THRIFT-2441: --- +1 > Cannot shutdown TThreadedServer when clients

[jira] [Commented] (THRIFT-2441) Cannot shutdown TThreadedServer when clients are still connected

[ https://issues.apache.org/jira/browse/THRIFT-2441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14487648#comment-14487648 ] Ben Craig commented on THRIFT-2441: --- oh, sorry about that. I wasn't sure if this had b

[jira] [Commented] (THRIFT-3084) C++ add concurrent client limit to threaded servers

[ https://issues.apache.org/jira/browse/THRIFT-3084?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14482250#comment-14482250 ] Ben Craig commented on THRIFT-3084: --- I was going to argue more, but then I argued mysel

[jira] [Commented] (THRIFT-3084) C++ add concurrent client limit to threaded servers

[ https://issues.apache.org/jira/browse/THRIFT-3084?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14482236#comment-14482236 ] Ben Craig commented on THRIFT-3084: --- I'm great with gutting TThreadedServer, but I don'

[jira] [Commented] (THRIFT-3084) C++ add concurrent client limit to threaded servers

[ https://issues.apache.org/jira/browse/THRIFT-3084?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14481443#comment-14481443 ] Ben Craig commented on THRIFT-3084: --- "That logic allows for maxConnections + 1 to be ac

[jira] [Commented] (THRIFT-3084) C++ add concurrent client limit to threaded servers

[ https://issues.apache.org/jira/browse/THRIFT-3084?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14481350#comment-14481350 ] Ben Craig commented on THRIFT-3084: --- I think that the ThreadManager class already provi

[jira] [Commented] (THRIFT-2441) Cannot shutdown TThreadedServer when clients are still connected

[ https://issues.apache.org/jira/browse/THRIFT-2441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14394837#comment-14394837 ] Ben Craig commented on THRIFT-2441: --- I don't know why the peek is there. In the server

[jira] [Commented] (THRIFT-1025) C++ ServerSocket should inherit from Socket with the necessary Ctor to listen on connections from a specific host (similar to perl library)

[ https://issues.apache.org/jira/browse/THRIFT-1025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14394729#comment-14394729 ] Ben Craig commented on THRIFT-1025: --- +1 > C++ ServerSocket should inherit from Socket

[jira] [Commented] (THRIFT-2441) Cannot shutdown TThreadedServer when clients are still connected

[ https://issues.apache.org/jira/browse/THRIFT-2441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14394389#comment-14394389 ] Ben Craig commented on THRIFT-2441: --- I approve of the direction of this patch. I lik

[jira] [Commented] (THRIFT-2441) Cannot shutdown TThreadedServer when clients are still connected

[ https://issues.apache.org/jira/browse/THRIFT-2441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14393112#comment-14393112 ] Ben Craig commented on THRIFT-2441: --- I guess I skipped a few items in my approach. Her

[jira] [Commented] (THRIFT-2441) Cannot shutdown TThreadedServer when clients are still connected

[ https://issues.apache.org/jira/browse/THRIFT-2441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14392880#comment-14392880 ] Ben Craig commented on THRIFT-2441: --- I think we're converging on a great answer. Thank

[jira] [Comment Edited] (THRIFT-2441) Cannot shutdown TThreadedServer when clients are still connected

[ https://issues.apache.org/jira/browse/THRIFT-2441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14392880#comment-14392880 ] Ben Craig edited comment on THRIFT-2441 at 4/2/15 4:00 PM: --- I t

[jira] [Commented] (THRIFT-2441) Cannot shutdown TThreadedServer when clients are still connected

[ https://issues.apache.org/jira/browse/THRIFT-2441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14391440#comment-14391440 ] Ben Craig commented on THRIFT-2441: --- "Did not provide a way to maintain the current beh

[jira] [Commented] (THRIFT-2441) Cannot shutdown TThreadedServer when clients are still connected

[ https://issues.apache.org/jira/browse/THRIFT-2441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14388768#comment-14388768 ] Ben Craig commented on THRIFT-2441: --- I agree that the issue is a real one, and should b

[jira] [Commented] (THRIFT-3063) C++ build -Wunused-parameter warnings on processor_test, TransportTest

[ https://issues.apache.org/jira/browse/THRIFT-3063?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14388564#comment-14388564 ] Ben Craig commented on THRIFT-3063: --- Visual Studio /W4 has a similar warning ( C4100 ).

[jira] [Commented] (THRIFT-2441) Cannot shutdown TThreadedServer when clients are still connected

[ https://issues.apache.org/jira/browse/THRIFT-2441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14388513#comment-14388513 ] Ben Craig commented on THRIFT-2441: --- TServerSocket's pattern is basically what I had in

[jira] [Commented] (THRIFT-2441) Cannot shutdown TThreadedServer when clients are still connected

[ https://issues.apache.org/jira/browse/THRIFT-2441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14386763#comment-14386763 ] Ben Craig commented on THRIFT-2441: --- I am leery in general of closing a transport in on

[jira] [Commented] (THRIFT-2441) Cannot shutdown TThreadedServer when clients are still connected

[ https://issues.apache.org/jira/browse/THRIFT-2441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14386696#comment-14386696 ] Ben Craig commented on THRIFT-2441: --- When there is a network disconnect, the socket han

[jira] [Commented] (THRIFT-2441) Cannot shutdown TThreadedServer when clients are still connected

[ https://issues.apache.org/jira/browse/THRIFT-2441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14386681#comment-14386681 ] Ben Craig commented on THRIFT-2441: --- "however this patch applies after the listening so

[jira] [Commented] (THRIFT-2441) Cannot shutdown TThreadedServer when clients are still connected

[ https://issues.apache.org/jira/browse/THRIFT-2441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14386093#comment-14386093 ] Ben Craig commented on THRIFT-2441: --- -1 for this specific fix. Closing a socket that i

[jira] [Commented] (THRIFT-2873) TPipe and TPipeServer don't compile on Windows with UNICODE enabled

[ https://issues.apache.org/jira/browse/THRIFT-2873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14234216#comment-14234216 ] Ben Craig commented on THRIFT-2873: --- +1 > TPipe and TPipeServer don't compile on Windo

[jira] [Commented] (THRIFT-2873) TPipe and TPipeServer don't compile on windows with UNICODE enabled

[ https://issues.apache.org/jira/browse/THRIFT-2873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14233606#comment-14233606 ] Ben Craig commented on THRIFT-2873: --- [~pascal.bach] I think you are correct. TPipe and

[jira] [Commented] (THRIFT-2873) TPipe and TPipeServer don't compile on windows with UNICODE enabled

[ https://issues.apache.org/jira/browse/THRIFT-2873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14233069#comment-14233069 ] Ben Craig commented on THRIFT-2873: --- casting from a const char * to a LPCTSTR with UNIC

[jira] [Commented] (THRIFT-2799) Build Problem(s): ld: library not found for -l:libboost_unit_test_framework.a

[ https://issues.apache.org/jira/browse/THRIFT-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14202050#comment-14202050 ] Ben Craig commented on THRIFT-2799: --- 'missing separator' normally means that you missed

[jira] [Commented] (THRIFT-2799) Build Problem(s): ld: library not found for -l:libboost_unit_test_framework.a

[ https://issues.apache.org/jira/browse/THRIFT-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14201005#comment-14201005 ] Ben Craig commented on THRIFT-2799: --- Try changing thrift / lib / cpp / test / Makefile

[jira] [Commented] (THRIFT-2799) Build Problem(s): ld: library not found for -l:libboost_unit_test_framework.a

[ https://issues.apache.org/jira/browse/THRIFT-2799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14200970#comment-14200970 ] Ben Craig commented on THRIFT-2799: --- A colon snuck into your library name. Your link l

[jira] [Commented] (THRIFT-2717) C++11 generator

[ https://issues.apache.org/jira/browse/THRIFT-2717?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14162264#comment-14162264 ] Ben Craig commented on THRIFT-2717: --- v2 headers could #error if used with too old of a

[jira] [Commented] (THRIFT-2717) C++11 generator

[ https://issues.apache.org/jira/browse/THRIFT-2717?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14161922#comment-14161922 ] Ben Craig commented on THRIFT-2717: --- Add some extra directories maybe? calculator.cpp #

[jira] [Commented] (THRIFT-2717) C++11 generator

[ https://issues.apache.org/jira/browse/THRIFT-2717?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14161901#comment-14161901 ] Ben Craig commented on THRIFT-2717: --- I agree that those are problems. I think that tho

[jira] [Commented] (THRIFT-2717) C++11 generator

[ https://issues.apache.org/jira/browse/THRIFT-2717?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14161861#comment-14161861 ] Ben Craig commented on THRIFT-2717: --- Most of the features mentioned in the ticket don't

[jira] [Commented] (THRIFT-2717) C++11 generator

[ https://issues.apache.org/jira/browse/THRIFT-2717?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14158101#comment-14158101 ] Ben Craig commented on THRIFT-2717: --- They only need to use tcxx::shared_ptr if they wan

[jira] [Commented] (THRIFT-2717) C++11 generator

[ https://issues.apache.org/jira/browse/THRIFT-2717?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14157949#comment-14157949 ] Ben Craig commented on THRIFT-2717: --- we may be able to (sort of) drop boost from the ex

[jira] [Commented] (THRIFT-2717) C++11 generator

[ https://issues.apache.org/jira/browse/THRIFT-2717?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14144771#comment-14144771 ] Ben Craig commented on THRIFT-2717: --- RHEL and CentOS 6 use gcc 4.4. RHEL 7 came out i

[jira] [Comment Edited] (THRIFT-2717) C++11 generator

[ https://issues.apache.org/jira/browse/THRIFT-2717?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14143739#comment-14143739 ] Ben Craig edited comment on THRIFT-2717 at 9/22/14 8:23 PM: I

  1   2   3   4   5   >