[ 
https://issues.apache.org/jira/browse/THRIFT-2441?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14492919#comment-14492919
 ] 

ASF GitHub Bot commented on THRIFT-2441:
----------------------------------------

Github user bufferoverflow commented on a diff in the pull request:

    https://github.com/apache/thrift/pull/424#discussion_r28270114
  
    --- Diff: lib/cpp/src/thrift/transport/TSocket.cpp ---
    @@ -493,29 +585,9 @@ uint32_t TSocket::read(uint8_t* buf, uint32_t len) {
           goto try_again;
         }
     
    -#if defined __FreeBSD__ || defined __MACH__
         if (errno_copy == THRIFT_ECONNRESET) {
    -      /* shigin: freebsd doesn't follow POSIX semantic of recv and fails 
with
    -       * THRIFT_ECONNRESET if peer performed shutdown
    -       * edhall: eliminated close() since we do that in the destructor.
    -       */
           return 0;
         }
    -#endif
    -
    -#ifdef _WIN32
    --- End diff --
    
    Thanks Jim, I'm fine with your explanations, CMake build on appveyor 
currently fails and I try to get it up and runnning again. I think Windows 
platform users shall use Visual Studio instead of cygwin and MinGW or as a 
better choice clang;-)



> Cannot shutdown TThreadedServer when clients are still connected
> ----------------------------------------------------------------
>
>                 Key: THRIFT-2441
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2441
>             Project: Thrift
>          Issue Type: Bug
>          Components: C++ - Library
>    Affects Versions: 0.9.1
>            Reporter: Chris Stylianou
>            Assignee: Ben Craig
>         Attachments: THRIFT-2441-prelim.patch
>
>
> When calling stop() on the TThreadedServer no interrupts are sent to the 
> client threads. This means the stop() call blocks on tasksMonitor.wait() 
> until all client naturally disconnect.
> How can we tell the client thread connections to close/exit during the 
> TThreadedServer::stop() call?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to