Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

disable EVENT_READ when waiting for EVENT_WRITE, and vice versa #117

Merged
merged 4 commits into from
Feb 7, 2017

Conversation

thinkingfish
Copy link
Contributor

This addresses #116

We are wiating only for either read or write event at any given point, so difficulty to write to socket will translate (eventually) to back pressure by ignoring incoming data from the same channel.

Yao Yue added 4 commits July 2, 2016 16:47
5b095bc Merge pull request twitter#126 from kevyang/kevyang/120
426d56a return NULL when cc_alloc/cc_realloc is called with size == 0
ad271d4 Merge pull request twitter#133 from kevyang/132
47dbdba suppress unused parameter warning in debug_log_flush
648d19e Merge pull request twitter#127 from kevyang/56
780941a Merge pull request twitter#130 from kevyang/129
b8af6c0 Merge pull request twitter#131 from kevyang/128
6ecc318 fix duplicate symbols in cc_signal
080c41d cc_array - stop doing arithmetic on void *
d526f7a add debug oriented memory management
a4fb927 Update bool member rules in style guide
05c6e1e explicitly make ccommon a C project to avoid checking for CXX related variables

git-subtree-dir: deps/ccommon
git-subtree-split: 5b095bc
@kevyang
Copy link
Contributor

kevyang commented Sep 27, 2016

ship it

@thinkingfish thinkingfish merged commit 58c42b4 into twitter:master Feb 7, 2017
@thinkingfish thinkingfish deleted the fix_event_write branch February 7, 2017 21:11
swlynch99 pushed a commit to swlynch99/pelikan-twitter that referenced this pull request Sep 30, 2019
use level-trigger in kqueue implementation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants