From 7b99863e027b5733c72001f3d51b0dee6e0fb01c Mon Sep 17 00:00:00 2001 From: cfc4n Date: Wed, 10 Jun 2020 14:59:11 +0800 Subject: [PATCH] mvcc: chanLen 1024 is to biger,and it used more memory. 128 seems to be enough. Sometimes the consumption speed is more than the production speed. See https://github.com/etcd-io/etcd/issues/11906 for more detail. --- mvcc/watchable_store.go | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/mvcc/watchable_store.go b/mvcc/watchable_store.go index 72c6b8be4..55d0e9dee 100644 --- a/mvcc/watchable_store.go +++ b/mvcc/watchable_store.go @@ -30,9 +30,8 @@ import ( var ( // chanBufLen is the length of the buffered chan // for sending out watched events. - // TODO: find a good buf value. 1024 is just a random one that - // seems to be reasonable. - chanBufLen = 1024 + // See https://github.com/etcd-io/etcd/issues/11906 for more detail. + chanBufLen = 128 // maxWatchersPerSync is the number of watchers to sync in a single batch maxWatchersPerSync = 512