changeset 30:c5b498c5aa3c

Keepalive: update comments, style.
author Maxim Dounin <mdounin@mdounin.ru>
date Wed, 31 Aug 2011 05:43:53 +0400
parents c7c4b2d68fdf
children 78fd926c306d
files ngx_http_upstream_keepalive_module.c
diffstat 1 files changed, 2 insertions(+), 15 deletions(-) [+]
line wrap: on
line diff
--- a/ngx_http_upstream_keepalive_module.c
+++ b/ngx_http_upstream_keepalive_module.c
@@ -318,19 +318,8 @@ ngx_http_upstream_free_keepalive_peer(ng
      * make sure that u->length is valid (we use u->header_sent flag to test
      * this).  Memcached is the only supported protocol for now.
      *
-     * Some notes on other possibilities (incomplete):
-     *
-     * fastcgi: u->pipe->upstream_done should be sufficient
-     *
-     * proxy buffered: u->pipe->upstream_done, 304 replies, replies to head
-     * requests (see RFC 2616, 4.4 Message Length)
-     *
-     * proxy unbuffered: 200 as for memcached (with u->length == 0 and
-     * header_sent), 304, replies to head requests
-     *
-     * subrequest_in_memory: won't work as of now
-     *
-     * TODO: move this logic to protocol modules (NGX_PEER_KEEPALIVE?)
+     * With experimental patches we are able to cache other connections as
+     * well.  Connection status is signalled via u->keepalive flag.
      */
 
     u = kp->upstream;
@@ -534,9 +523,7 @@ ngx_http_upstream_keepalive(ngx_conf_t *
     for (i = 2; i < cf->args->nelts; i++) {
 
         if (ngx_strcmp(value[i].data, "single") == 0) {
-
             kcf->single = 1;
-
             continue;
         }