Skip to content

Backport #11920 to 20.5: Fix using current database while checking access rights.#11963

Merged
alesapin merged 223 commits into20.5from
backport/20.5/11920
Jul 2, 2020
Merged

Backport #11920 to 20.5: Fix using current database while checking access rights.#11963
alesapin merged 223 commits into20.5from
backport/20.5/11920

Conversation

@abyss7
Copy link
Contributor

@abyss7 abyss7 commented Jun 25, 2020

Original pull-request #11920
Cherry-pick pull-request #11956

alesapin and others added 30 commits June 11, 2020 18:55
After the main test, run queries from `website.xml` in parallel using
`clickhouse-benchmark`. This can be useful to test the effects of
concurrency on performance. Comparison test can miss some effects
because it always runs queries sequentially, and many of them are even
single-threaded.
This PR adds a server level config for overriding the default max number
of threads in global thread pool that is currently allowed (10,000).

This might be useful in scenarios where there are a large number of
distributed queries that are executing concurrently and where the
default number of max threads might not be necessarily be sufficient.
This adds the `max_thread_pool_size` config to
tests/server-test.xml file.
This adds the docs for the new server level setting
`max_thread_pool_size`.
alexey-milovidov and others added 25 commits June 24, 2020 16:14
They keep removing the old image every couple of weeks, and the entire
docker update fails.
In case of OSX jemalloc register itself as a default zone allocator.

But when you link statically then zone_register() will not be called,
and even will be optimized out:

  $ nm clickhouse.patched  | grep -c zone_register
  0

Fix this, by manually calling it.

v2: extern C
Fix jemalloc under OSX (by registering it as default zone explicitly)
Add concurrent benchmark to performance test
Put clickhouse-local data to /tmp by default
…t-database

Fix using current database while checking access rights.
…f7a4489166d9ef4db68ac7d528287

Cherry pick #11920 to 20.5: Fix using current database while checking access rights.
@abyss7 abyss7 added the pr-backport Changes, backported to release branch. Do not use manually - automated use only! label Jun 25, 2020
@alesapin alesapin merged commit e0cf236 into 20.5 Jul 2, 2020
@alesapin alesapin deleted the backport/20.5/11920 branch July 2, 2020 14:43
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.