aboutsummaryrefslogtreecommitdiff
path: root/databases/clickhouse/files/patch-src_Interpreters_AsynchronousMetrics.cpp
diff options
context:
space:
mode:
authorRuslan Makhmatkhanov <rm@FreeBSD.org>2020-09-10 15:21:26 +0000
committerRuslan Makhmatkhanov <rm@FreeBSD.org>2020-09-10 15:21:26 +0000
commit5ac71a446cf8860129777cf7f0f72990dcf8dbc5 (patch)
tree310fa6f9a2a7d66a0359f044d480ddedcaca2f1c /databases/clickhouse/files/patch-src_Interpreters_AsynchronousMetrics.cpp
parent6385c18d93f05ce3e19750ead71b5d06ba0a057e (diff)
downloadports-5ac71a446cf8860129777cf7f0f72990dcf8dbc5.tar.gz
ports-5ac71a446cf8860129777cf7f0f72990dcf8dbc5.zip
databases/clickhouse: apply patch from #13859: MemoryTracking support
Clickhouse for FreeBSD updating no memory usage stats. In the case build without NDEBUG, allocator use 4KB chunk, those dont account when freeing and MemoryTracking metric grow up to ~6TB, after this clickhouse stop query processing with error "Memory limit (total) exceeded: would use 6.24 TiB" due skipping calc allocation < 4MB Patch still not upstreamed, obtained from: https://github.com/ClickHouse/ClickHouse/issues/13859 + https://github.com/ClickHouse/ClickHouse/pull/13869 reported and tested by: Vyacheslav (via Clickhouse telegram) PR: 249205 Submitted by: olevole@olevole.ru (maintainer) Obtained from: clickhouse repository
Notes
Notes: svn path=/head/; revision=548207
Diffstat (limited to 'databases/clickhouse/files/patch-src_Interpreters_AsynchronousMetrics.cpp')
-rw-r--r--databases/clickhouse/files/patch-src_Interpreters_AsynchronousMetrics.cpp11
1 files changed, 11 insertions, 0 deletions
diff --git a/databases/clickhouse/files/patch-src_Interpreters_AsynchronousMetrics.cpp b/databases/clickhouse/files/patch-src_Interpreters_AsynchronousMetrics.cpp
new file mode 100644
index 000000000000..0f7ea8b8ce75
--- /dev/null
+++ b/databases/clickhouse/files/patch-src_Interpreters_AsynchronousMetrics.cpp
@@ -0,0 +1,11 @@
+--- src/Interpreters/AsynchronousMetrics.cpp.orig 2020-08-31 16:22:57 UTC
++++ src/Interpreters/AsynchronousMetrics.cpp
+@@ -194,7 +194,7 @@ void AsynchronousMetrics::update()
+ new_values["Uptime"] = context.getUptimeSeconds();
+
+ /// Process memory usage according to OS
+-#if defined(OS_LINUX)
++#if defined(OS_LINUX) || defined(OS_FREEBSD)
+ {
+ MemoryStatisticsOS::Data data = memory_stat.get();
+