From 9c105090928bcd3955c68d26b787dc324c9d42d6 Mon Sep 17 00:00:00 2001 From: Patrick Donnelly Date: Thu, 6 Sep 2018 08:52:25 -0700 Subject: [PATCH] log: use smaller buffer for ConcreteEntry This brings down the static size of the memory used by the logging infrastructure: If we used 1024, we'd have 1088*10000 = 10880000 = 10MB in use by the ring buffer and 2*1088*100 = 2*108800 = 2*106KB for the m_new and m_flush vectors. In my testing, 1024 covers most log entries. Note, I've kept 4096 for the StackStringStream via MutableEntry as these are already allocated on the heap and cached in a thread local vector. Generally there should only be about a dozen of these allocated so it's worth keeping a larger buffer. Signed-off-by: Patrick Donnelly --- src/log/Entry.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/log/Entry.h b/src/log/Entry.h index 076512cbe0c..2753b1942b5 100644 --- a/src/log/Entry.h +++ b/src/log/Entry.h @@ -107,7 +107,7 @@ public: } private: - boost::container::small_vector str; + boost::container::small_vector str; }; }