Skip to content

Query Condition Cache: Reduce lock contention#87337

Closed
zhongyuankai wants to merge 3 commits intoClickHouse:masterfrom
zhongyuankai:reduce_lock_for_qcc
Closed

Query Condition Cache: Reduce lock contention#87337
zhongyuankai wants to merge 3 commits intoClickHouse:masterfrom
zhongyuankai:reduce_lock_for_qcc

Conversation

@zhongyuankai
Copy link
Copy Markdown
Contributor

@zhongyuankai zhongyuankai commented Sep 19, 2025

Follow-up to #86076

Changelog category (leave one):

  • Performance Improvement

Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):

For entries with the same key already present in the cache, clone the entry and cache it into the QueryConditionCacheWriter to avoid unnecessary updates and reduce lock contention.

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@zhongyuankai
Copy link
Copy Markdown
Contributor Author

@rschu1ze Please help review if you have time.

@rschu1ze
Copy link
Copy Markdown
Member

@rschu1ze Please help review if you have time.

Didn't happen yet, sorry.

@rschu1ze rschu1ze self-assigned this Sep 23, 2025
@rschu1ze rschu1ze added the can be tested Allows running workflows for external contributors label Sep 23, 2025
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Sep 23, 2025

Workflow [PR], commit [8144a48]

@clickhouse-gh clickhouse-gh bot added the pr-performance Pull request with some performance improvements label Sep 23, 2025
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Oct 28, 2025

Dear @rschu1ze, this PR hasn't been updated for a while. You will be unassigned. Will you continue working on it? If so, please feel free to reassign yourself.

@rschu1ze rschu1ze closed this Dec 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

can be tested Allows running workflows for external contributors pr-performance Pull request with some performance improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants