[
https://issues.apache.org/jira/browse/THRIFT-2881?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14238488#comment-14238488
]
Hudson commented on THRIFT-2881:
--------------------------------
SUCCESS: Integrated in Thrift #1395 (See
[https://builds.apache.org/job/Thrift/1395/])
THRIFT-2881 Handle errors from Accept() correctly (jensg: rev
57cd421de3b681b4e401dcc5d965671362663998)
* lib/go/thrift/simple_server.go
> Handle errors from Accept() correctly
> -------------------------------------
>
> Key: THRIFT-2881
> URL: https://issues.apache.org/jira/browse/THRIFT-2881
> Project: Thrift
> Issue Type: Bug
> Components: Go - Library
> Affects Versions: 0.9.2
> Environment: Linux Amazon AMI
> Reporter: awaw fumin
> Assignee: Jens Geyer
> Labels: patch
> Fix For: 0.9.3
>
> Original Estimate: 24h
> Remaining Estimate: 24h
>
> Often in tests, servers are started and closed when a test case is finished,
> as in the standard library's "httptest" package.
> http://golang.org/pkg/net/http/httptest/#example_Server
> However, currently the "p.quit" channel of TSimpleServer does not correctly
> handle the case when the listener socket is Closed, resulting in spurious
> logs with the text "Accept err: ...".
> The pull request that fixes this is https://github.com/apache/thrift/pull/309
> .
> For details of the discussion behind handling closed listener sockets, please
> see
> https://code.google.com/p/go/issues/detail?id=4373
> http://zhen.org/blog/graceful-shutdown-of-go-net-dot-listeners/
> http://stackoverflow.com/a/13419724/2182406
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
