Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport #12182 to 20.5: Cap max_memory_usage* limits to the process resident memory #12367

Merged
merged 1,105 commits into from
Jul 10, 2020

Conversation

robot-clickhouse
Copy link
Member

Original pull-request #12182
Cherry-pick pull-request #12351

This pull-request is a last step of an automated backporting.
Treat it as a standard pull-request: look at the checks and resolve conflicts.
Merge it only if you intend to backport changes to the target branch, otherwise just close it.

alesapin and others added 30 commits July 4, 2020 10:35
* Add unbundled mode flag

* Move skip lists to clickhouse-test

* More verbose message about skip

* Make skip-list optional

* Update clickhouse-test
By default IMPORTED target has a scope in the directory in which it is created
and below. This leads to the following issues when building UNBUNDLED:

Target "clickhouse" links to target "Poco::Data::ODBC" but the target was not
found.  Perhaps a find_package() call is missing for an IMPORTED target, or an
ALIAS target is missing?
* simplify reading in order of sorting key

* add perf test for reading many parts

* Revert "simplify reading in order of sorting key"

This reverts commit 7267d7c.

* add threshold for preliminary merge for reading in order

* better threshold

* limit threads in test
Added a showcase of minimal Docker image
Minor cmake fixes for UNBUNDLED build
Fix Parquet for LowCardinality
Fix result_rows and result_bytes metrics for selects.
akuzm and others added 25 commits July 8, 2020 14:25
Added a test for PaddedPODArray just in case
AST-based query fuzzing mode for clickhouse-client
Headers for CSVWithNames in StorageFile
…ary-failed-to-load

Show error after TrieDictionary failed to load.
* first try

* change submodule

* Update .gitmodules

* include build directory

* Update .gitmodules

Co-authored-by: Nikita Mikhailov <jakalletti@jakalletti-build.sas.yp-c.yandex.net>
Bump arrow to 0.17 (and flatbuffers to v1.12, required by arrow)
Cap max_memory_usage* limits to the process resident memory
…79c23a28db217d66d20e59f98d769

Cherry pick #12182 to 20.5: Cap max_memory_usage* limits to the process resident memory
@robot-clickhouse robot-clickhouse added the pr-backport Changes, backported to release branch. Do not use manually - automated use only! label Jul 9, 2020
@alexey-milovidov alexey-milovidov merged commit f79a2de into 20.5 Jul 10, 2020
@alexey-milovidov alexey-milovidov deleted the backport/20.5/12182 branch July 10, 2020 05:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-backport Changes, backported to release branch. Do not use manually - automated use only!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet