You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2012/04/27 17:27:55 UTC
git commit: TS-1209 Allow for background fill when producer is a
transform.
Updated Branches:
refs/heads/master f36afeb92 -> 04311e0c6
TS-1209 Allow for background fill when producer is a transform.
Author: Robert Logue.
Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/04311e0c
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/04311e0c
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/04311e0c
Branch: refs/heads/master
Commit: 04311e0c6e9b256ea8129b5a3cd5ccfbd786c381
Parents: f36afeb
Author: Robert Logue <rp...@hotmail.co.uk>
Authored: Fri Apr 27 09:23:42 2012 -0600
Committer: Leif Hedstrom <zw...@apache.org>
Committed: Fri Apr 27 09:23:42 2012 -0600
----------------------------------------------------------------------
CHANGES | 3 +++
proxy/http/HttpSM.cc | 4 +++-
2 files changed, 6 insertions(+), 1 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/04311e0c/CHANGES
----------------------------------------------------------------------
diff --git a/CHANGES b/CHANGES
index 4895d51..d56e86f 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,5 +1,8 @@
-*- coding: utf-8 -*-
Changes with Apache Traffic Server 3.1.4
+ *) [TS-1209] Allow for background fill even when a transform plugin is
+ the producer. Author: Robert Logue.
+
*) [TS-1229] clean up RecordsCofing.cc, remove unused entries.
*) [TS-1142] record ram hit in stats.
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/04311e0c/proxy/http/HttpSM.cc
----------------------------------------------------------------------
diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 94b8c72..4542244 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -2911,7 +2911,9 @@ HttpSM::is_bg_fill_necessary(HttpTunnelConsumer * c)
// There must be another consumer for it to worthwhile to
// set up a background fill
- if (c->producer->num_consumers > 1 && c->producer->vc_type == HT_HTTP_SERVER && c->producer->alive == true) {
+ if (c->producer->num_consumers > 1 &&
+ (c->producer->vc_type == HT_HTTP_SERVER || c->producer->vc_type == HT_TRANSFORM) &&
+ c->producer->alive == true) {
// If threshold is 0.0 or negative then do background
// fill regardless of the content length. Since this
// is floating point just make sure the number is near zero