Skip to content

Add system.rocksdb and read rocksdb/rocksdb_TABLE options from config#26821

Merged
kssenii merged 2 commits intoClickHouse:masterfrom
azat:rocksdb-improvements
Jul 27, 2021
Merged

Add system.rocksdb and read rocksdb/rocksdb_TABLE options from config#26821
kssenii merged 2 commits intoClickHouse:masterfrom
azat:rocksdb-improvements

Conversation

@azat
Copy link
Copy Markdown
Member

@azat azat commented Jul 26, 2021

Changelog category (leave one):

  • Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Expose rocksdb statistics via system.rocksdb table. Read rocksdb options from ClickHouse config (rocksdb/rocksdb_TABLE keys)

Cc: @sundy-li
Follow-up for: #15073

@robot-clickhouse robot-clickhouse added the pr-improvement Pull request with some product improvements label Jul 26, 2021
@azat azat marked this pull request as draft July 26, 2021 18:44
@azat azat force-pushed the rocksdb-improvements branch from d9b0f36 to 96ebfb8 Compare July 26, 2021 19:24
@azat azat marked this pull request as ready for review July 26, 2021 19:24
v2: fix USE_ROCKSDB check (there is #cmakedefine01 so it is either 0/1,
so we should not check with #ifdef)
@azat azat force-pushed the rocksdb-improvements branch from 96ebfb8 to a21c49a Compare July 26, 2021 20:14
@kssenii kssenii self-assigned this Jul 26, 2021
@azat azat force-pushed the rocksdb-improvements branch 3 times, most recently from 3ef887f to 3801605 Compare July 27, 2021 05:13
@azat azat marked this pull request as draft July 27, 2021 05:23
v2: Cover rocksdb options in ClickHouse config
v3: add missing __init__.py
v4: Rework rocksdb options from config
v5: add column_family_options support
@azat azat force-pushed the rocksdb-improvements branch from 3801605 to 8ef677b Compare July 27, 2021 05:41
@azat azat marked this pull request as ready for review July 27, 2021 05:41
Copy link
Copy Markdown
Member

@kssenii kssenii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@azat
Copy link
Copy Markdown
Member Author

azat commented Jul 27, 2021

Integration tests (asan) — fail: 34, passed: 1536, flaky: 0
Integration tests (release) — fail: 34, passed: 946, flaky: 3
Integration tests (thread) — fail: 34, passed: 184, flaky: 0

@kssenii kssenii merged commit b5dce17 into ClickHouse:master Jul 27, 2021
@azat azat deleted the rocksdb-improvements branch July 28, 2021 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-improvement Pull request with some product improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants