You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by su...@apache.org on 2016/03/05 04:28:15 UTC

[2/2] trafficserver git commit: update docs.

update docs.


Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/6991aecd
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/6991aecd
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/6991aecd

Branch: refs/heads/master
Commit: 6991aecd9cc0385cf5f76e0e0e444e8eb220f176
Parents: 2378e8a
Author: Sudheer Vinukonda <su...@yahoo-inc.com>
Authored: Sat Mar 5 03:28:03 2016 +0000
Committer: Sudheer Vinukonda <su...@yahoo-inc.com>
Committed: Sat Mar 5 03:28:03 2016 +0000

----------------------------------------------------------------------
 plugins/experimental/collapsed_forwarding/README             | 8 ++++++++
 .../collapsed_forwarding/collapsed_forwarding.cc             | 8 ++++++++
 2 files changed, 16 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/6991aecd/plugins/experimental/collapsed_forwarding/README
----------------------------------------------------------------------
diff --git a/plugins/experimental/collapsed_forwarding/README b/plugins/experimental/collapsed_forwarding/README
index 4aa2686..6cdd5c5 100644
--- a/plugins/experimental/collapsed_forwarding/README
+++ b/plugins/experimental/collapsed_forwarding/README
@@ -18,6 +18,14 @@
 // proxy.config.http.background_fill_active_timeout      0 /////////////////////////
 // proxy.config.http.background_fill_completed_threshold 0 /////////////////////////
 ////////////////////////////////////////////////////////////////////////////////////
+// Additionally, given that collapsed forwarding works based on cache write
+// lock failure detection, the plugin requires cache to be enabled and ready.
+// On a restart, Traffic Server typically takes a few seconds to initialize
+// the cache depending on the cache size and number of dirents. While the
+// cache is not ready yet, collapsed forwarding can not detect the write lock
+// contention and so can not work. The setting proxy.config.http.wait_for_cache
+// may be enabled which allows blocking incoming connections from being
+// accepted until cache is ready.
 ////////////////////////////////////////////////////////////////////////////////////
 // This plugin currently supports only per-remap mode activation.
 ////////////////////////////////////////////////////////////////////////////////////

http://git-wip-us.apache.org/repos/asf/trafficserver/blob/6991aecd/plugins/experimental/collapsed_forwarding/collapsed_forwarding.cc
----------------------------------------------------------------------
diff --git a/plugins/experimental/collapsed_forwarding/collapsed_forwarding.cc b/plugins/experimental/collapsed_forwarding/collapsed_forwarding.cc
index 18861c7..3a40e4c 100644
--- a/plugins/experimental/collapsed_forwarding/collapsed_forwarding.cc
+++ b/plugins/experimental/collapsed_forwarding/collapsed_forwarding.cc
@@ -39,6 +39,14 @@
 // proxy.config.http.background_fill_active_timeout      0 /////////////////////////
 // proxy.config.http.background_fill_completed_threshold 0 /////////////////////////
 ////////////////////////////////////////////////////////////////////////////////////
+// Additionally, given that collapsed forwarding works based on cache write
+// lock failure detection, the plugin requires cache to be enabled and ready.
+// On a restart, Traffic Server typically takes a few seconds to initialize
+// the cache depending on the cache size and number of dirents. While the
+// cache is not ready yet, collapsed forwarding can not detect the write lock
+// contention and so can not work. The setting proxy.config.http.wait_for_cache
+// may be enabled which allows blocking incoming connections from being
+// accepted until cache is ready.
 ////////////////////////////////////////////////////////////////////////////////////
 // This plugin currently supports only per-remap mode activation.
 ////////////////////////////////////////////////////////////////////////////////////