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

Wip 10194 master #3125

Merged
merged 2 commits into from Jan 29, 2015
Merged

Wip 10194 master #3125

merged 2 commits into from Jan 29, 2015

Conversation

yehudasa
Copy link
Member

@yehudasa yehudasa commented Dec 9, 2014

A more intrusive fix to 10194. This will reuse connection structure between requests.

@yehudasa
Copy link
Member Author

@liewegas fixed stuff, will need to squash after review

@ceph-jenkins
Copy link
Collaborator

Can one of the admins OK this pull request so I can run a build?

@yehudasa
Copy link
Member Author

@liewegas ping?

@liewegas
Copy link
Member

looks good to me.. squash and merge?

@liewegas liewegas added this to the hammer milestone Jan 29, 2015
yehudasa and others added 2 commits January 29, 2015 10:09
A queue that provides multi-readers, multi-writers concurrency with a
reduced lock contention.

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
Fixes: #10194

Reuse the fcgx connection structure. It forces FCGX_Accept_r() freeing
up or reusing the connection if possible. This is needed for certain
fastcgi module (mod-proxy-fcgi) that sets the FCGI_KEEP_CONN, which
prevents the connection from being closed when calling FCGX_Finish_r().

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
@loic-bot
Copy link

SUCCESS: the output of run-make-check.sh on centos-centos7 for 029c7de is http://paste2.org/_PnW1Z5c9

:octocat: Sent from GH.

liewegas added a commit that referenced this pull request Jan 29, 2015
Wip 10194 master

Reviewed-by: Sage Weil <sage@redhat.com>
@liewegas liewegas merged commit 243860b into master Jan 29, 2015
@liewegas liewegas deleted the wip-10194-master branch January 29, 2015 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants