GitHub user bgedik reopened a pull request:
https://github.com/apache/thrift/pull/1476
Remove premature call to workSocket() in TNonblockingServer
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/bgedik/thrift
bugfix/non-blocking-server-calls-workSocket-too-early
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/thrift/pull/1476.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1476
----
commit f3c682e9c41d67d8cd1ab2d75f6259b6e0af40da
Author: Bugra Gedik <bgedik@...>
Date: 2018-01-21T17:43:49Z
remove workSocket call that is too early
commit e115634a2a9844d08a6d017859e3d838414d2194
Author: Bugra Gedik <bgedik@...>
Date: 2018-01-22T00:35:31Z
Make the work socket conditional
commit 8e5ba48c9d62ee4ae79aac5f2cc4f98d2f535866
Author: Bugra Gedik <bgedik@...>
Date: 2018-01-22T01:03:04Z
Revert back the changes
commit 017d242479c985a9cb476bcd2da22abaa62ed052
Author: Bugra Gedik <bgedik@...>
Date: 2018-01-22T18:52:33Z
Only call workSocket() when there is pending data to read
commit c929047d4a43c3969cc40b204e3717877667edb2
Author: Bugra Gedik <bgedik@...>
Date: 2018-01-22T19:29:12Z
Minor update to a comment
commit fb72b94b513f665e49488d8aaee7eda3f4b57a94
Author: Bugra Gedik <bgedik@...>
Date: 2018-01-22T20:51:55Z
Only call workSocket() when there is pending data to read
commit 4777cab48641d23bd2246248a07a7a4249a3709f
Author: Bugra Gedik <bgedik@...>
Date: 2018-01-22T21:43:04Z
Only call workSocket() when there is pending data to read
commit f69e91be02275c8a06f489030fcda24449329ce2
Author: Bugra Gedik <bgedik@...>
Date: 2018-01-22T23:44:10Z
Fix the CMake build
commit 84051190db3234468ffed11a2a61ab8ba6d7e1fd
Author: Bugra Gedik <bgedik@...>
Date: 2018-01-22T23:49:27Z
Fix the CMake build
commit 7b60db3a5535195ec6096fea8ead757266d716b5
Author: Bugra Gedik <bgedik@...>
Date: 2018-01-22T23:58:34Z
Fix the CMake build
commit 081096091cc828dd7dbcf0962cdaa6c791317e0c
Author: Bugra Gedik <bgedik@...>
Date: 2018-01-23T00:14:09Z
Use the correct type for ioctlsocket call
commit 501c4401ddb93cf9bf451dd289d4c3768c0a7b15
Author: Bugra Gedik <bgedik@...>
Date: 2018-01-24T19:19:57Z
Make non-blocking peek optional
commit a9dc3f118acb820189696bd13735444ad2d129a3
Author: Bugra Gedik <bgedik@...>
Date: 2018-01-24T20:46:40Z
Revert "Make non-blocking peek optional"
This reverts commit 501c4401ddb93cf9bf451dd289d4c3768c0a7b15.
commit c8d422a633a6b848982c48cb769285906f2ea027
Author: Bugra Gedik <bgedik@...>
Date: 2018-01-24T21:12:35Z
Review fixes
commit d64a8e0d4709ece4086f054fae7c1c98e3a0ca77
Author: Bugra Gedik <bgedik@...>
Date: 2018-01-24T21:15:14Z
Review fixes
commit 83d3c13f09238831e2d06e8b67a018acbcc656c4
Author: Bugra Gedik <bgedik@...>
Date: 2018-01-24T21:16:28Z
Review fixes
----
---