[
https://issues.apache.org/jira/browse/THRIFT-1304?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13093114#comment-13093114
]
Hudson commented on THRIFT-1304:
--------------------------------
Integrated in Thrift #242 (See [https://builds.apache.org/job/Thrift/242/])
THRIFT-1304. cpp: TNonblockingServer: pass in the connection context to
process()
Patch: Adam Simpkins
bryanduxbury : http://svn.apache.org/viewvc/?view=rev&rev=1162943
Files :
* /thrift/trunk/lib/cpp/src/server/TNonblockingServer.cpp
> TNonblockingServer: pass in the connection context to
> -------------------------------------------------------
>
> Key: THRIFT-1304
> URL: https://issues.apache.org/jira/browse/THRIFT-1304
> Project: Thrift
> Issue Type: Improvement
> Components: C++ - Library
> Reporter: Dave Watson
> Assignee: Dave Watson
> Priority: Minor
> Fix For: 0.8
>
> Attachments:
> 0023-TNonblockingServer-pass-in-the-connection-context-to.patch
>
>
> From aa5cdf5b40e046b41a0c61e521dba0e78e1887b0 Mon Sep 17 00:00:00 2001
> From: Adam Simpkins <[email protected]>
> Date: Sat, 17 Apr 2010 01:10:15 +0000
> Subject: [PATCH 23/33] TNonblockingServer: pass in the connection context to
> process()
> Summary:
> When using TNonblockingServer without a ThreadManager, it always passed
> NULL to process(), instead of passing in the context returned by
> TProcessorEventHandler::getContext(). This was broken and could cause
> TProcessorEventHandlers to crash, since they expected to get the value
> they had previously returned via getContext().
> Test Plan:
> Ran the test code in [fb unittest]. The non-threaded TNonblockingServer tests
> no longer crash.
> Revert Plan:
> OK
> ---
> lib/cpp/src/server/TNonblockingServer.cpp | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
