tests: optimize stateless tests to fit into 5G of memory#93362
tests: optimize stateless tests to fit into 5G of memory#93362azat wants to merge 17 commits intoClickHouse:masterfrom
Conversation
…_offsets Previously SELECT can eat up to 20GiB!
|
Workflow [PR], commit [175cec3] Summary: ❌
|
|
Integration tests are way less approachable. For example, there was not a single case when integration tests ran successfully on my machine. We need fewer integration tests, not more - e.g., all tests for Iceberg over S3 could be converted to functional. |
|
Totally agree, I do not like to run the either (I think I was the first who introduced running separate clickhouse-server in stateless tests - 3afa946) But those tests exceeds memory limits, and also the logic to start server in the stateless tests is complex. I'm OK to mark them as no-parallel and increase memory limit instead, it will work for now, but not sure it is better. Later, we can think about other options - provide some helpers to run server in stateless tests or make integration tests better, though there is a better way - just run |
|
Ok, actually the change is acceptable, let's proceed... |
|
OK, let me merge this and rebase #82361 on top |
|
@antaljanosbenjamin likely it will be merged as #82361 |
Changelog category (leave one):
This is part of #82361, since it may requires changes to prakitka that may take sometime, while this contains enough changes on it's own
Changes:
clickhouse-clientin parallel - converted to usecurlclickhouse-server- converted to integration tests