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
- Set LICENSE_FILE as it is not a boilerplate
- Hook provided tests to our framework, after
converting them to modern Python version
- Bump port revisions of two dependent ports,
to chase the shared library version bump
Reported by: portscout
Maintainer reset due to the extended timeout, as per policy.
Take maintainership.
7 ports are bumped.
PR: 239367
Approved by: waitman@waitman.net (maintainer's timeout; ~6months on a PR#235789 for the same port)
as defined in Mk/bsd.default-versions.mk which has moved from GCC 8.3
to GCC 9.1 under most circumstances now after revision 507371.
This includes ports
- with USE_GCC=yes or USE_GCC=any,
- with USES=fortran,
- using Mk/bsd.octave.mk which in turn features USES=fortran, and
- with USES=compiler specifying openmp, nestedfct, c11, c++0x, c++11-lang,
c++11-lib, c++14-lang, c++17-lang, or gcc-c++11-lib
plus, everything INDEX-11 shows with a dependency on lang/gcc9 now.
PR: 238330
* Clickhouse requires now a compiler that understands C++17.
While I'm here:
* Set the TEST option as non-default and convert all runtime dependencies of
that option to default ones. The former description was somewhat
misleading because the TEST option only installed two additional Python
scripts.
The test query datasets in %%DATADIR%%-test/ and the previously mentioned
Python scripts were always installed regardless if TEST was set/unset
because they are part of the normal release installation. This also would
lead to build failures if the TEST option was disabled. [1]
* Use the TEST option for compiling and testing of additional tests by
enabling -DENABLE_TESTS (instead of introducing a new option with another
name for it). [1]
* Also mark the TEST option as broken when building on FreeBSD 13.0-CURRENT
with Clang 8 because it segfaults with an assertion error in function
"getZExtValue()".
* Pet portlint a bit by placing some variables to their intended locations.
Changelog:
https://github.com/yandex/ClickHouse/blob/v19.5.3.8-stable/CHANGELOG.md
PR: 237318
Submitted by: proler@gmail.com (maintainer)
Approved by: maintainer [1], mentors (implicit)
While in devel/double-conversion, order USExx clauses, and replace CMAKE_POSITION_INDEPENDENT_CODE with BUILD_SHARED_LIBS since shared libs should be a default
Update and bump databases/clickhouse
Approved by: portmgr blanket
Ports that build out of source now simply can use "USES=cmake"
instead of "USES=cmake:outsource". Ports that fail to build
out of source now need to specify "USES=cmake:insource".
I tried to only set insource where explictely needed.
PR: 232038
Exp-run by: antoine
defined via Mk/bsd.default-versions.mk which has moved from GCC 7.4 t
GCC 8.2 under most circumstances.
This includes ports
- with USE_GCC=yes or USE_GCC=any,
- with USES=fortran,
- using Mk/bsd.octave.mk which in turn features USES=fortran, and
- with USES=compiler specifying openmp, nestedfct, c11, c++0x, c++11-lang,
c++11-lib, c++14-lang, c++17-lang, or gcc-c++11-lib
plus, as a double check, everything INDEX-11 showed depending on lang/gcc7.
PR: 231590
Simplify some ports where DragonFlyBSD no longer needs to be special-cased.
Submitted by: rene
Reviewed by: bapt, jbeich
Differential Revision: https://reviews.freebsd.org/D17724
