You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Dave Watson (JIRA)" <ji...@apache.org> on 2011/09/09 02:18:08 UTC

[jira] [Created] (THRIFT-1335) Add accept timeout to TServerSocket

 Add accept timeout to TServerSocket

-------------------------------------

                 Key: THRIFT-1335
                 URL: https://issues.apache.org/jira/browse/THRIFT-1335
             Project: Thrift
          Issue Type: Improvement
          Components: C++ - Library
            Reporter: Dave Watson
            Priority: Minor
         Attachments: 0031-Improve-pusher_serverrr.patch

>From d07e47a771d9cca8111b2773c8e3436a5e9442ae Mon Sep 17 00:00:00 2001
From: Wei Chen <we...@fb.com>
Date: Wed, 21 Apr 2010 21:12:34 +0000
Subject: [PATCH 31/33] Add accept timeout to TServerSocket

Summary:
Added an accept timeout to the TServerSocket poll loop

Test Plan:
I run the servers in testing rack and verified it works both for intra rack and
inter rack transfer.

scottstraw, simpkins, weichen, thrift-team@lists
Revert Plan:
OK

---
 lib/cpp/src/transport/TServerSocket.cpp |   12 +++++++++++-
 lib/cpp/src/transport/TServerSocket.h   |    3 +++
 2 files changed, 14 insertions(+), 1 deletions(-)



--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Closed] (THRIFT-1335) Add accept timeout to TServerSocket

Posted by "Jake Farrell (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/THRIFT-1335?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jake Farrell closed THRIFT-1335.
--------------------------------

       Resolution: Fixed
    Fix Version/s: 0.8

Committed, thanks for the patch.

>  Add accept timeout to TServerSocket

> -------------------------------------
>
>                 Key: THRIFT-1335
>                 URL: https://issues.apache.org/jira/browse/THRIFT-1335
>             Project: Thrift
>          Issue Type: Improvement
>          Components: C++ - Library
>            Reporter: Dave Watson
>            Priority: Minor
>             Fix For: 0.8
>
>         Attachments: 0031-Improve-pusher_serverrr.patch
>
>
> From d07e47a771d9cca8111b2773c8e3436a5e9442ae Mon Sep 17 00:00:00 2001
> From: Wei Chen <we...@fb.com>
> Date: Wed, 21 Apr 2010 21:12:34 +0000
> Subject: [PATCH 31/33] Add accept timeout to TServerSocket
> Summary:
> Added an accept timeout to the TServerSocket poll loop
> Test Plan:
> I run the servers in testing rack and verified it works both for intra rack and
> inter rack transfer.
> scottstraw, simpkins, weichen, thrift-team@lists
> Revert Plan:
> OK
> ---
>  lib/cpp/src/transport/TServerSocket.cpp |   12 +++++++++++-
>  lib/cpp/src/transport/TServerSocket.h   |    3 +++
>  2 files changed, 14 insertions(+), 1 deletions(-)

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (THRIFT-1335) Add accept timeout to TServerSocket

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/THRIFT-1335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13104369#comment-13104369 ] 

Hudson commented on THRIFT-1335:
--------------------------------

Integrated in Thrift #264 (See [https://builds.apache.org/job/Thrift/264/])
    THRIFT-1335: Add accept timeout to TServerSocket
Client: cpp
Patch: Dave Watson

Added an accept timeout to the TServerSocket poll loop

jfarrell : http://svn.apache.org/viewvc/?view=rev&rev=1170443
Files : 
* /thrift/trunk/lib/cpp/src/transport/TServerSocket.cpp
* /thrift/trunk/lib/cpp/src/transport/TServerSocket.h


>  Add accept timeout to TServerSocket

> -------------------------------------
>
>                 Key: THRIFT-1335
>                 URL: https://issues.apache.org/jira/browse/THRIFT-1335
>             Project: Thrift
>          Issue Type: Improvement
>          Components: C++ - Library
>            Reporter: Dave Watson
>            Priority: Minor
>             Fix For: 0.8
>
>         Attachments: 0031-Improve-pusher_serverrr.patch
>
>
> From d07e47a771d9cca8111b2773c8e3436a5e9442ae Mon Sep 17 00:00:00 2001
> From: Wei Chen <we...@fb.com>
> Date: Wed, 21 Apr 2010 21:12:34 +0000
> Subject: [PATCH 31/33] Add accept timeout to TServerSocket
> Summary:
> Added an accept timeout to the TServerSocket poll loop
> Test Plan:
> I run the servers in testing rack and verified it works both for intra rack and
> inter rack transfer.
> scottstraw, simpkins, weichen, thrift-team@lists
> Revert Plan:
> OK
> ---
>  lib/cpp/src/transport/TServerSocket.cpp |   12 +++++++++++-
>  lib/cpp/src/transport/TServerSocket.h   |    3 +++
>  2 files changed, 14 insertions(+), 1 deletions(-)

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (THRIFT-1335) Add accept timeout to TServerSocket

Posted by "Hudson (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/THRIFT-1335?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13143950#comment-13143950 ] 

Hudson commented on THRIFT-1335:
--------------------------------

Integrated in Thrift #316 (See [https://builds.apache.org/job/Thrift/316/])
    THRIFT-1335: Add accept timeout to TServerSocket
Client: cpp
Patch: Dave Watson

FIX: add accTimout_ Initialization to DomainSocket Constructor

roger : http://svn.apache.org/viewvc/?view=rev&rev=1197523
Files : 
* /thrift/trunk/lib/cpp/src/transport/TServerSocket.cpp

                
>  Add accept timeout to TServerSocket

> -------------------------------------
>
>                 Key: THRIFT-1335
>                 URL: https://issues.apache.org/jira/browse/THRIFT-1335
>             Project: Thrift
>          Issue Type: Improvement
>          Components: C++ - Library
>            Reporter: Dave Watson
>            Priority: Minor
>             Fix For: 0.8
>
>         Attachments: 0031-Improve-pusher_serverrr.patch
>
>
> From d07e47a771d9cca8111b2773c8e3436a5e9442ae Mon Sep 17 00:00:00 2001
> From: Wei Chen <we...@fb.com>
> Date: Wed, 21 Apr 2010 21:12:34 +0000
> Subject: [PATCH 31/33] Add accept timeout to TServerSocket
> Summary:
> Added an accept timeout to the TServerSocket poll loop
> Test Plan:
> I run the servers in testing rack and verified it works both for intra rack and
> inter rack transfer.
> scottstraw, simpkins, weichen, thrift-team@lists
> Revert Plan:
> OK
> ---
>  lib/cpp/src/transport/TServerSocket.cpp |   12 +++++++++++-
>  lib/cpp/src/transport/TServerSocket.h   |    3 +++
>  2 files changed, 14 insertions(+), 1 deletions(-)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (THRIFT-1335) Add accept timeout to TServerSocket

Posted by "Dave Watson (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/THRIFT-1335?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Dave Watson updated THRIFT-1335:
--------------------------------

    Attachment: 0031-Improve-pusher_serverrr.patch

>  Add accept timeout to TServerSocket

> -------------------------------------
>
>                 Key: THRIFT-1335
>                 URL: https://issues.apache.org/jira/browse/THRIFT-1335
>             Project: Thrift
>          Issue Type: Improvement
>          Components: C++ - Library
>            Reporter: Dave Watson
>            Priority: Minor
>         Attachments: 0031-Improve-pusher_serverrr.patch
>
>
> From d07e47a771d9cca8111b2773c8e3436a5e9442ae Mon Sep 17 00:00:00 2001
> From: Wei Chen <we...@fb.com>
> Date: Wed, 21 Apr 2010 21:12:34 +0000
> Subject: [PATCH 31/33] Add accept timeout to TServerSocket
> Summary:
> Added an accept timeout to the TServerSocket poll loop
> Test Plan:
> I run the servers in testing rack and verified it works both for intra rack and
> inter rack transfer.
> scottstraw, simpkins, weichen, thrift-team@lists
> Revert Plan:
> OK
> ---
>  lib/cpp/src/transport/TServerSocket.cpp |   12 +++++++++++-
>  lib/cpp/src/transport/TServerSocket.h   |    3 +++
>  2 files changed, 14 insertions(+), 1 deletions(-)

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira