Skip to content

Commit addeeb0

Browse files
committed
docs: clarify predictive_preconnect_ratio upper limit
Add documentation explaining that predictive_preconnect_ratio is limited to 3. Per_upstream_preconnect_ratio and predictive_preconnect_ratio are both limited to 3, but previously only the former had this upper bound documented.
1 parent f62f929 commit addeeb0

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

api/envoy/config/cluster/v3/cluster.proto

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -748,6 +748,9 @@ message Cluster {
748748
// If both this and preconnect_ratio are set, Envoy will make sure both predicted needs are met,
749749
// basically preconnecting max(predictive-preconnect, per-upstream-preconnect), for each
750750
// upstream.
751+
//
752+
// This is limited somewhat arbitrarily to 3 because preconnecting too aggressively can
753+
// harm latency more than the preconnecting helps.
751754
google.protobuf.DoubleValue predictive_preconnect_ratio = 2
752755
[(validate.rules).double = {lte: 3.0 gte: 1.0}];
753756
}

0 commit comments

Comments
 (0)